-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Conversation
Conflicts: src/thirdparty/CodeMirror2
…clean-up-for-dark-ui
@redmunds see my comment above regarding the arrows. |
@LarZ I saw your comment about not being able to style the triangle, so I was expecting to see the default one. I'm not sure if people will realize that's a selectable list. You should demo this (and all your other changes) to the team in tomorrow's standup to see what everyone else thinks. |
@larz0 For the triangle, maybe this only works on Windows, but this seems to right-align an image in <sleect> element (needs margin on right, just a test image):
|
@redmunds I've fixed the input error border. Regarding the triangle, using @mackenza that should be fixed now, thanks! |
@LarZ |
@larz0 confirmed menus are OK now when switching between dark and light and back again. Using your latest changes, the js parameter hints are not readable. light grey text on white background. |
@redmunds isn't 7px the y value? |
@mackenza could you take a screenshot? |
"each <percentage> or <length> represents an offset and must be preceded by a keyword, which specifies from which edge the offset is given. For example, ‘background-position: bottom 10px right 20px’ represents a ‘10px’ vertical offset up from the bottom edge and a ‘20px’ horizontal offset leftward from the right edge. If three values are given, the missing offset is assumed to be zero. " http://www.w3.org/TR/css3-background/#the-background-position |
@larz0 here you go... |
@redmunds that's awesome, it worked! I had no idea it works like that. |
Looks great. Merging. |
@larz0, nice work on this, beautiful :). One question for you. I am running Mac OSX, disable all extensions, with and without theme scrollbars preference checked, and my editor vertical and horizontal scrollbars are still white and gray. Is this by design? Because they don't seem to match the rest of the dark UI: |
@lkcampbell Those are the old OS X scrollbars and that's what they look like, go to System Preferences -> General and set "Show scroll bars" to "Automatically based on…" to get the new quiet scroll bars but I have a feeling you don't want that. The new scroll bars are hard to see on dark themes because for some reason the native scroll bar isn't aware of editor's background; the color should be inverted when the background is dark like how scroll bar colors are inverted on dark websites. This can be solved by customizing the dark scroll bar for Mac (like Windows and Linux) and that should also affect the appearance of old scrollbars. I'm happy to do that but I'll need consensus because the behavior is slightly different, quiet scroll bars disappear when the content is not scrolling and some people prefer that. Hope that makes sense. |
@larz0, yes that makes sense. I still get the white and gray scrollbars when I use "Automatically..." but I am sure that is just a configuration issue. I see the quiet scrollbar when I set it to "When scrolling" and I do see what you mean about the quiet scrollbar being very difficult to see. Seems like neither choice is great but the gray on white is at least more practical since you can see it a lot better. |
@pthiess This one won't show up on Waffle because it's closed. If more testing is required, this is one of those cases where a separate tracking issue is required in order to show up in Waffle. |
https://trello.com/c/GjJNtQSX/1381-more-thorough-dark-mode