Skip to content
This repository has been archived by the owner on Feb 17, 2022. It is now read-only.

Added discrete custom scrollbars. #40

Merged
merged 1 commit into from
Oct 7, 2017
Merged

Added discrete custom scrollbars. #40

merged 1 commit into from
Oct 7, 2017

Conversation

carloskelly13
Copy link

@carloskelly13 carloskelly13 commented Oct 5, 2017 •
edited
Loading

Addresses #39

  • Add discrete custom scrollbars to the log, modules, assets and problem containers
  • Ensure the main scrollbar is still hidden

screen shot 2017-10-05 at 2 37 01 pm

cc @kenwheeler

@carloskelly13 carloskelly13 added this to the 1.0 milestone Oct 5, 2017
ryan-roemer
ryan-roemer approved these changes Oct 6, 2017
View reviewed changes
Copy link
Member

@ryan-roemer ryan-roemer left a comment

Choose a reason for hiding this comment

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

Code LGTM. Need a tire kicking too?

@carloskelly13
Copy link
Author

@ryan-roemer Nah, it’s pretty cosmetic unless you want to try it out

@ryan-roemer
Copy link
Member

@carlospaelinck -- Just go for it 😄

@carloskelly13 carloskelly13 merged commit 5f91c21 into master Oct 7, 2017
@carloskelly13 carloskelly13 deleted the custom-scrollbars branch October 7, 2017 02:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ryan-roemer ryan-roemer ryan-roemer approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@carloskelly13 @ryan-roemer