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

New mixed titleBarStyle #57117

Closed
Logerfo opened this issue Aug 23, 2018 · 9 comments
Closed

New mixed titleBarStyle #57117

Logerfo opened this issue Aug 23, 2018 · 9 comments
Assignees
Labels
feature-request Request for new features or functionality menus Menu items and widget issues *out-of-scope Posted issue is not in scope of VS Code titlebar VS Code main title bar issues
Milestone

Comments

@Logerfo
Copy link
Contributor

Logerfo commented Aug 23, 2018

As I described in #16363 (comment), changing the title bar style in order to achieve dark menus made me lost the ability to hide the title bar. I wish there was an third option, "mixed" (you can name it whatever you'd like), which uses the custom renderer (the one which achieves the dark theme), but keeps the native title bar, so I can hide it with the window flag. This implies in making the menu bar parallel to the title bar, not inside it.
Would this be hard to implement? Thanks.

@vscodebot
Copy link

vscodebot bot commented Aug 23, 2018

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

@sbatten sbatten added feature-request Request for new features or functionality workbench-menu titlebar VS Code main title bar issues labels Aug 26, 2018
@sbatten sbatten added this to the Backlog milestone Aug 26, 2018
@Guema
Copy link

Guema commented Oct 25, 2018

I think the better way to do this should be to make a new custom title bar top menu that looks like the actual native one.
As it is the electron's chromium menu, it is not rendered in the web layout, but in a wrapping os-driven layout that is just not displayed at all when you set custom title bar.
For me, proceeding like this would be as better as it is easier :

  • could be themable
  • will match more Code brand
  • is consistent with the "custom" choice for titlebar.

@Logerfo
Copy link
Contributor Author

Logerfo commented Oct 25, 2018

When the custom title bar was implemented only in the insider preview, when you selected the custom option, the title menu was inside the title bar if it's always shown (like it still does as today) or in a new bar below the title bar if it's only shown when requested (which does not happen today, as it stills shows the menus inside the custom title bar). As for what I'm proposing, the latter is what we need for the new configuration, but using the native title bar instead of the custom one.

@Logerfo
Copy link
Contributor Author

Logerfo commented Oct 25, 2018

@Guema if the title bar is not native, then it's not hideable through the windows flag, which is the idea of this proposal.

@Guema
Copy link

Guema commented Oct 25, 2018

I understand what you request, i was just technically speaking.
Electron, the framework that run Vscode, is - by default - rendering a first basic os-driven window with a second basic chromium-driven top menu (the one that is used when title-bar is set to native) (and the chromium dev console eventually). You can find exactly the same menubar on other electron app like : slack, ryver for exemple.
So there is 2 layout, plus a third, the one that render your web content.

Electron can be configured to not display this default layout: native title bar and top menu, and then display only the web content (custom mode). It is still curiously possible to display console. but i presume it's placement is not managed by Vscode codebase directly, but chromium layout system

What i suppose is, with Electron, it will be difficult or even impossible to place the default chromium menubar under the custom title bar. I don't even know if it is possible to show this menubar without using native layout. (unless they tweak Electron's code i presume)

What i said is if they want to make a second mode where menu bar is under title bar instead of inside it, devs will probably have to imitate the native one in their code.

@Logerfo
Copy link
Contributor Author

Logerfo commented Oct 25, 2018

@Guema

it will be difficult or even impossible to place the default chromium menubar under the custom title bar

I understand, but that is exactly the opposite of what I suggested, which is: place the custom menu bar below the native title bar.
As I can see for my previous superficial knowledge and your explanation about electron rendering, in order to achieve what I requested, the native mode should be used, with the custom menus rendered in the web content and without using the native menu bar (that's why I called it "mixed" mode).

@Guema
Copy link

Guema commented Oct 25, 2018

Ho, ok.

Well i didn't understood this like that.
Hum. This other particular case should not be really hard to make. from what i see about the app

@sbatten sbatten added menus Menu items and widget issues *out-of-scope Posted issue is not in scope of VS Code and removed workbench-menu labels Oct 7, 2019
@vscodebot
Copy link

vscodebot bot commented Oct 10, 2019

This issue is being closed to keep the number of issues in our inbox on a manageable level, we are closing issues that are not going to be addressed in the foreseeable future: We look at the number of votes the issue has received and the number of duplicate issues filed. More details here. If you disagree and feel that this issue is crucial: We are happy to listen and to reconsider.

If you wonder what we are up to, please see our roadmap and issue reporting guidelines.

Thanks for your understanding and happy coding!

@vscodebot vscodebot bot closed this as completed Oct 10, 2019
@Logerfo
Copy link
Contributor Author

Logerfo commented Oct 10, 2019

With the menu bar inside the title bar, I don't see a reason to develop what the issue requests and agree with the decision to drop it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality menus Menu items and widget issues *out-of-scope Posted issue is not in scope of VS Code titlebar VS Code main title bar issues
Projects
None yet
Development

No branches or pull requests

3 participants