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

Fixed #5732, #5735: fixed box-shadow and background-color in quick-open menu #5733

Merged
merged 2 commits into from
Jul 23, 2019

Conversation

fangnx
Copy link

@fangnx fangnx commented Jul 16, 2019

  • This patch added dark-theme support for the box-shadow on the quick-open menu. To be aligned with VS Code and the current UI, the top divider (shadow) would be black in the dark theme and light-grey in the light theme.
  • Also, aligned background-color of the quick-open menu with other widgets (e.g. the side-panel widget). This would also make the top shadow line more visible in the dark mode.

Before

theia-quickopen-line

After

image

@fangnx fangnx force-pushed the fnx-fix-5732 branch 2 times, most recently from d6a9362 to 7eaa191 Compare July 16, 2019 20:28
…pport dark theme

- This patch added dark-theme support for the `box-shadow` on the quick-open menu. To be aligned with VS Code and the current UI, the top divider (shadow) would be black in the dark theme and light-grey in the light theme.

Signed-off-by: fangnx <naxin.fang@ericsson.com>
… with other widgets

- Aligned the background color of quick-open menu with other widgets (e.g. the side-panel widget). This would also make the top shadow line more visible in the dark mode.

Signed-off-by: fangnx <naxin.fang@ericsson.com>
@fangnx fangnx changed the title Fixed #5732: fixed box-shadow in quick-open menu to support dark theme Fixed #5732, #5735: fixed box-shadow and background-color in quick-open menu Jul 17, 2019
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

It looks much better now thank you! 👍

@vince-fugnitto
Copy link
Member

vince-fugnitto commented Jul 23, 2019

@jbicker would you like to take a look as well or shall I merge? :)

@jbicker
Copy link
Contributor

jbicker commented Jul 23, 2019

merge :)

@vince-fugnitto vince-fugnitto merged commit ff8b013 into eclipse-theia:master Jul 23, 2019
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.

3 participants