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

preferences: Fix problems.decorations.tabbar.enabled #9059

Merged
merged 1 commit into from
Feb 16, 2021

Conversation

DucNgn
Copy link
Contributor

@DucNgn DucNgn commented Feb 12, 2021

What it does

  • Add a check to properly enable/disable tab bar overlay icon when toggling problems.decorations.tabbar.enabled.

How to test

  • Open editors, make some errors or warnings. Observe that the markers are displayed on tab bar
    ie:
    image

  • Open preferences, toggle the option problems.decorations.tabbar.enabled.

  • Observe that the markers disappeared on tab bar.
    image

Review checklist

Reminder for reviewers

Signed-off-by: Duc Nguyen [email protected]

+ Add a check to properly enable/disable tab bar overlay icon when toggling `problems.decorations.tabbar.enabled`.

Signed-off-by: Duc Nguyen <[email protected]>
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.

I confirmed the issue on master, and how the pull-request fixes the behavior.

@vince-fugnitto vince-fugnitto added the markers issues related to problem markers label Feb 12, 2021
@vince-fugnitto vince-fugnitto merged commit f3a15db into eclipse-theia:master Feb 16, 2021
@vince-fugnitto vince-fugnitto deleted the dn/tabbarEnable branch February 16, 2021 18:11
@paul-marechal paul-marechal added this to the 1.11.0 milestone Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
markers issues related to problem markers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants