Skip to content

support hiding the status bar#10092

Merged
vince-fugnitto merged 4 commits intoeclipse-theia:masterfrom
EstherPerelman:toggle-status-bar
Sep 29, 2021
Merged

support hiding the status bar#10092
vince-fugnitto merged 4 commits intoeclipse-theia:masterfrom
EstherPerelman:toggle-status-bar

Conversation

@EstherPerelman
Copy link
Contributor

@EstherPerelman EstherPerelman commented Sep 14, 2021

Signed-off-by: Esther Perelman esther.perelman@sap.com

What it does

This PR supports hiding the status bar (blue bar on the bottom) in 2 ways:

  1. By selecting the Toggle status Bar command from the view menu:
    image

  2. By updating the workbench.statusBar.visible setting in the preferences file.
    image

How to test

  • Validate that the status bar appears for the first time,
  • Play around with the 2 ways mentions above...

Review checklist

Reminder for reviewers

@EstherPerelman EstherPerelman force-pushed the toggle-status-bar branch 3 times, most recently from 392465a to 47e6eb9 Compare September 14, 2021 10:22
@msujew msujew added the statusbar issues related to the statusbar label Sep 14, 2021
@EstherPerelman
Copy link
Contributor Author

@vince-fugnitto can you help me to solve the issue causes the builds to fail, The problem is with the injection of the PreferenceService to the status-bar.tsx I guess I need to change some code in theia\packages\core\src\browser\keybinding.spec.ts but I don't know how to do it (I'm not so familiar with inversify)
Actually I used the corePreferences but changed to PreferenceService during the tries to solve this problem, It will be better if you give me a solution for it... Thanks in advance!

@vince-fugnitto
Copy link
Member

@vince-fugnitto can you help me to solve the issue causes the builds to fail...

@EstherPerelman you can try the following file to resolve the test bindings:

https://gist.github.com/vince-fugnitto/19456daac6167c2f3ec71626050ba1a7

@EstherPerelman
Copy link
Contributor Author

@EstherPerelman you can try the following file to resolve the test bindings:

https://gist.github.com/vince-fugnitto/19456daac6167c2f3ec71626050ba1a7

@vince-fugnitto thank you for the clear and perfect solution!

Copy link
Member

@msujew msujew 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 and I can confirm that everything works as expected:

  • Toggling the status bar using the menu entry/command/keyboard shortcut
  • Toggling the status bar using the preferences

@EstherPerelman
Copy link
Contributor Author

@vince-fugnitto, Thank you for the comments, I've uploaded the fixes and replied to this comment, Please merge it if possible...

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 functionality of the changes work well for me 👍
I have a minor comment about adding a changelog entry.

@EstherPerelman
Copy link
Contributor Author

The functionality of the changes work well for me 👍
I have a minor comment about adding a changelog entry.

@vince-fugnitto thank you, I have updated the CHANGELOG.md

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 confirmed that:

  • the preference works well
  • using the command properly toggles the statusbar and updates the preference

@EstherPerelman
Copy link
Contributor Author

EstherPerelman commented Sep 29, 2021

The changes look good to me 👍 I confirmed that:

  • the preference works well
  • using the command properly toggles the statusbar and updates the preference

@vince-fugnitto This PR is quite necessary for us, If it will be merged for tomorrow it will help us (this not seem to be so big change, don't you think like?)

@vince-fugnitto vince-fugnitto merged commit d9f8bc8 into eclipse-theia:master Sep 29, 2021
@vince-fugnitto vince-fugnitto added this to the 1.18.0 milestone Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

statusbar issues related to the statusbar

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants