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

GH-7834: Restored the find widget's "border" color. #7835

Merged
merged 1 commit into from
May 19, 2020

Conversation

kittaakos
Copy link
Contributor

--theia-input-border as the outline-color had no effect: #7340 (comment)

Fixes #7834

Signed-off-by: Akos Kitta kittaakos@typefox.io

What it does

Fixes the border color of the find/replace widget.

How to test

Try the find/replace widget in the editor, set the focus, compare the "border" color with the master.

Review checklist

Reminder for reviewers

`--theia-input-border` as the `outline-color` had no effect:
#7340 (comment)

Fixes #7834

Signed-off-by: Akos Kitta <kittaakos@typefox.io>
@akosyakov akosyakov requested a review from vince-fugnitto May 19, 2020 06:27
@akosyakov akosyakov added monaco issues related to monaco theming issues related to theming labels May 19, 2020
@akosyakov akosyakov requested a review from lmcbout May 19, 2020 06:27
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.

The changes look good to me, I verified using multiple builtin themes and the border for the search/find widget was always properly themed.

@kittaakos
Copy link
Contributor Author

Thank you for the verification, @vince-fugnitto 👍

@kittaakos kittaakos merged commit 46255a2 into master May 19, 2020
@kittaakos kittaakos deleted the kittaakos/monaco-the-find-replace-7834 branch May 19, 2020 13:10
Copy link
Contributor

@lmcbout lmcbout left a comment

Choose a reason for hiding this comment

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

Tested on UBUNTU 18.04
Same Blue border color apply now in Electron, Chrome and Firefox.

@kittaakos
Copy link
Contributor Author

Thanks for lot for checking it on another OS too, @lmcbout 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
monaco issues related to monaco theming issues related to theming
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[monaco] The find/replace widget has odd border color
4 participants