ignism / nucleus-dark-ui

Atom UI theme
MIT License
112 stars 34 forks source link

Scroll bar #89

Closed onetrev closed 7 years ago

onetrev commented 7 years ago

Pretty sure this just happened with a recent update, but the scroll bar is now old school looking for me.

image

scippio commented 7 years ago

Yeah, last update did it

joecorkerton commented 7 years ago

Really don't like the new scroll bar, is there a way to revert back to the old one?

ignism commented 7 years ago

This must be an issue with the deprecation of the Shadow DOM, in preparation for 1.13.0 I removed all ::shadow and /deep/ instances. I will try to fix this, although I don't have this issue on OS X. For now a quick fix could be to rollback to 0.10.15

@onetrev Do you run Atom Beta, does it have the same problem in the latest version there?

onetrev commented 7 years ago

I don't run Atom Beta, but installed it so I could check for you. And yes, confirmed the scroll bar is back to good/normal in Atom Beta.

MichaelAquilina commented 7 years ago

Can confirm this issue shows in the current stable release of Atom and is fixed when using the beta

a-tsvil commented 7 years ago

Have same on windows 10. Atom v1.12.5.

McAllaster commented 7 years ago

@ignism I am having this issue on OSX El Capitan 10.11.6

Edit: a simple fix right now (as I am using minimap as my scrollbar) is to disable scrollbars inside of the atom-text-editor::shadow element inside of your style.less with the following:

.vertical-scrollbar, .horizontal-scrollbar, .scrollbar-corner { display: none; }

ignism commented 7 years ago

I rolled back to a previous commit. just before I removed the shadow dom, which will be standard from version 1.13.0 and higher. But for now I the previous commit will work. Future patches will have a >= 1.13 in the engine section of the package.json to avoid further problems.