Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make scroll bars wider #7060

Merged
merged 3 commits into from Apr 17, 2018
Merged

Make scroll bars wider #7060

merged 3 commits into from Apr 17, 2018

Conversation

sts10
Copy link
Contributor

@sts10 sts10 commented Apr 7, 2018

I think the scroll bars on Mastodon's front-end could be a bit wider than 8px. I probably didn't execute this exactly right, but I tried to bump it up to 12px in basics.scss (I admittedly did not spend a lot of time trying to figure out which scss file).

Also, I couldn't figure out how to test this cuz I couldn't figure out how to install Ruby 2.5.1 on my machine :(

So in general this is not a great PR haha. But my request still stands -- wider scroll bars would be nice.

Should overwrite the setting in reset.scss. This is untested at this point.
Set scrollbars to 12 px wide rather than 8px
@@ -125,3 +125,8 @@ button {
outline: 0 !important;
}
}

::-webkit-scrollbar {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sts10
Copy link
Contributor Author

sts10 commented Apr 11, 2018

Sorry, bit confused by your request for changes. Do you want (a) me to change the width and height to 12px in the reset.scss file rather than in the basics.scss file or (b) remove scrollbar height and width specifications altogether.

I think (a) is a strange move-- generally css reset files serve the purpose of zeroing out or normalizing properties so that ever user has the same defaults, reducing browser inconsistencies. Hence why, in my original PR, I chose to "overwrite" the 8px in basics.scss.

Thus, in these new changes, I assumed you meant I should take route (b)-- remove the scrollbar height and width specifications altogether. My hope here is that browsers will fall back on some sort of system setting, which will like be wide enough to click and drag. This could even be a good move from an accessibility stand point.

Let me know if that was not the change you were thinking of.

@Gargron Gargron merged commit 1171545 into mastodon:master Apr 17, 2018
@sts10
Copy link
Contributor Author

sts10 commented Apr 19, 2018

Psyched that this got merged! My first contribution to Mastodon!

I'm assuming you've tested this on a few browsers and it looks good? I'm only worried cuz I have yet to get Mastodon to run locally (due to unrelated Ruby gem problems on my home machine), and I can't quite produce it via Chrome's Inspect Element.

@nightpool
Copy link
Member

yeah i think this broke scrollbars on chrome

@sts10
Copy link
Contributor Author

sts10 commented Apr 19, 2018

Ack, that's what I was worried about.

Maybe we should just change the hard-coded 8px to 12px, either in basics.scss or reset.scss, rather than removing the specification altogether.

beatrix-bitrot added a commit to beatrix-bitrot/glitch.social that referenced this pull request Apr 19, 2018
Gargron pushed a commit that referenced this pull request Apr 19, 2018
* Revert "Make scroll bars wider (#7060)"

This reverts commit 1171545.

* for real make webkit scrollbar wider
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.

None yet

3 participants