-
Notifications
You must be signed in to change notification settings - Fork 106
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
Update: Wider thumbnails #963
Conversation
Verified that @ConradJChan has signed the CLA. Thanks for the pull request! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
@@ -55,7 +55,6 @@ $thumbnail-sidebar-width: 201px; | |||
flex: 1 0 auto; | |||
overflow: hidden; | |||
padding: 0; | |||
width: 134px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this not needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I don't think so because the flexbox width seems to take precedence
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I take it back, ie11 needs this, but I'll set it to the flex-basis instead
…-preview into wider-thumbnails
35bf279
Increases the thumbnails sidebar width by 25px. Also decreases the selected state border to 3px
Before:
![Screen Shot 2019-03-25 at 3 08 32 PM](https://user-images.githubusercontent.com/17791289/54957902-545e1100-4f11-11e9-9c7e-cc0682b549d4.png)
Now:
![Screen Shot 2019-03-25 at 3 07 52 PM](https://user-images.githubusercontent.com/17791289/54957913-588a2e80-4f11-11e9-8596-ace0d190e645.png)