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

The verbose status bar #9

Closed
oshliaer opened this issue Nov 24, 2020 · 8 comments
Closed

The verbose status bar #9

oshliaer opened this issue Nov 24, 2020 · 8 comments

Comments

@oshliaer
Copy link

oshliaer commented Nov 24, 2020

I think that the status bar of the extension is redurant.

1606202625530

As you can see it is placed over a sufficiently long space.

My suggestion is to reduce the status bar to a few characters. For example

  1. Sync Scroll NORMAL
  2. Sync Scroll OFFSET
  3. Sync Scroll OFF

Also if this can be view I think if the Sync Scroll NORMAL/OFFSET is activated then Sync Scroll should be enabled programmatically.

Please review the issue and let me know what you think and hat you prefered in a PR (if there is a need for it).

@dqisme
Copy link
Owner

dqisme commented Nov 24, 2020

I think that the status bar of the extension is redurant.

1606202625530

As you can see it is placed over a sufficiently long space.

My suggestion is to reduce the status bar to a few characters. For example

  1. Sync Scroll NORMAL
  2. Sync Scroll OFFSET
  3. Sync Scroll OFF

Also if this can be view I think if the Sync Scroll NORMAL/OFFSET is activated then Sync Scroll should be enabled programmatically.

Please review the issue and let me know what you think and hat you prefered in a PR (if there is a need for it).

That's really a concise idea. I like it. Actually I was tangling about the long status bar and two switches for a while.
Thanks for your advice. I would try to implement it and keep you updated if there's any improvement.

@moxwel
Copy link

moxwel commented Mar 23, 2021

Also... Could have a setting to hide the status bar commands too?

@dqisme
Copy link
Owner

dqisme commented Apr 3, 2021

Also... Could have a setting to hide the status bar commands too?

Hey @moxwel, thanks for the advice. For clarifying your requirement, does it mean that you want to have a setting to hide the items in the status bar but leaving the commands so that the mode can be switched in the "Show All Commands" menu or shortcuts?

@moxwel
Copy link

moxwel commented Apr 3, 2021

Also... Could have a setting to hide the status bar commands too?

Hey @moxwel, thanks for the advice. For clarifying your requirement, does it mean that you want to have a setting to hide the items in the status bar but leaving the commands so that the mode can be switched in the "Show All Commands" menu or shortcuts?

Exactly! Only hide the items in the status bar.

@lonix1
Copy link

lonix1 commented Mar 20, 2022

Hiding the status bar items would be nice. But an even better option would be:

{
  "synchronizedScrolling.statusbarItems": "show",  // or "hide" or "showWhenEnabled"
}

The showWhenEnabled options would show the statusbar item, ONLY when sync scrolling is enabled. That is the most sensible option in most cases. You don't use this extension all the time, so most of the time the statusbar would be "clean", and would notify you that sync scrolling is enabled only when you're using it.

And of course you could hide the items with hide.

@dqisme
Copy link
Owner

dqisme commented Dec 7, 2022

Hi @contributorpw after so long time, a new patch version (v1.3.1) published, please check it out. It took your advice that merge the two menubar items into one. Now it only has one button to change the different modes with the three options 😃

@dqisme
Copy link
Owner

dqisme commented Dec 7, 2022

Hiding the status bar items would be nice. But an even better option would be:

{
  "synchronizedScrolling.statusbarItems": "show",  // or "hide" or "showWhenEnabled"
}

The showWhenEnabled options would show the statusbar item, ONLY when sync scrolling is enabled. That is the most sensible option in most cases. You don't use this extension all the time, so most of the time the statusbar would be "clean", and would notify you that sync scrolling is enabled only when you're using it.

And of course you could hide the items with hide.

Hi @lonix1 , thanks for your advice, for now there is always a workaround that you can disable the extension in your vscode and all the related items will not display.

@dqisme dqisme closed this as completed Dec 7, 2022
@oshliaer
Copy link
Author

@dqisme ,

thanks so much!

I'm using this ext sice microsoft/vscode#33110 (comment)

That's cool!

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

No branches or pull requests

4 participants