Skip to content

Conversation

@Thann
Copy link
Contributor

@Thann Thann commented Dec 4, 2014

Hey guys, what do you think of this?

@Mottie
Copy link
Member

Mottie commented Dec 4, 2014

The scrollbar code needs to be moved above the

/* inline code blocks - selector group is completed by the theme file */

section because it breaks the additional syntax highlighting selectors.

I moved it locally, and added an !important flag after each setting and I don't see the scroll bar getting styled in neither Chrome, nor Firefox... at least on a Windows platform.

@silverwind
Copy link
Member

I don't really think we should style the scrollbar, there are a lot of dedicated styles on userstyles for that, it's a user preference I wouldn't wanna make in this style. And getting it right cross-platform and cross-browser is hard :)

Mottie added a commit that referenced this pull request Dec 4, 2014
@Thann
Copy link
Contributor Author

Thann commented Dec 4, 2014

Yeah, I agree with "doing it right is hard", that's why I wanted to get some preliminary temperature readings. I'll close the PR then =]

@silverwind
Copy link
Member

We could add a separate repo for a (dark) scrollbar style. I once tried to mimic a OSX-style scrollbar for Firefox on Windows:

https://github.com/silverwind/overlay-scrollbars/blob/master/overlay-scrollbars.css

I think it was rather buggy, but could be a starting point :)

@Thann
Copy link
Contributor Author

Thann commented Dec 5, 2014

Cool, thanks, I'll play around with this over the weekend and try to get some simple multi-platform prototype.

@Thann Thann closed this Dec 5, 2014
@Thann Thann deleted the scrollbar branch December 5, 2014 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants