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

Mac: allow for more custom title bar styles #15098

Closed
bpasero opened this issue Nov 7, 2016 · 20 comments
Closed

Mac: allow for more custom title bar styles #15098

bpasero opened this issue Nov 7, 2016 · 20 comments
Assignees
Labels
feature-request Request for new features or functionality ux User experience issues
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Nov 7, 2016

On Mac OS, modern applications typically take over control of the title bar. Some examples:

Skype for Business:

image

Outlook:

image

VS Code so far is not leveraging this and thus the title appears like this:

image

Proposed solution introduces a new setting window.titleBarStyle that allows to switch to a custom title bar (and potentially also to a inline style as discussed in #12628).

image

image

@bpasero bpasero added feature-request Request for new features or functionality ux User experience issues workbench labels Nov 7, 2016
@bpasero bpasero added this to the November 2016 milestone Nov 7, 2016
@bpasero bpasero self-assigned this Nov 7, 2016
@smlombardi
Copy link

Will this setting be in Insiders build soon?

@bpasero
Copy link
Member Author

bpasero commented Nov 7, 2016

@smlombardi tomorrow 👍

@bgashler1
Copy link
Contributor

bgashler1 commented Nov 7, 2016

@bpasero this is great. It's close to what I was planning on proposing for a conservative first step. Thanks! The only differences were choice of colors for the title bar (I matched the tab well and divided it up using a muted horizontal line) and a tighter grouping of the the traffic lights to fit within the width of the activity bar.
screen shot 2016-11-07 at 7 35 20 am

@bpasero
Copy link
Member Author

bpasero commented Nov 7, 2016

@bgashler1 great. I was about to ask if you have ideas for which coloring to use. I just picked the ones for dark, light and high contrast theme.

They live in https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/browser/parts/titlebar/media/titlebarpart.css#L55

@jtlowe
Copy link

jtlowe commented Nov 7, 2016

Could this feature be wrapped up in an extension instead? This could give more flexibility to customize the color of the titlebar.

@bpasero
Copy link
Member Author

bpasero commented Nov 8, 2016

We are not planning to give extensions a way to put stuff there. The main reason is that this feature is not available on Linux and Windows at the moment.

@jtlowe
Copy link

jtlowe commented Nov 8, 2016

OK. Then how about a more specific name for the settings property, so it is easily recognized as applying to mac only?

@bpasero
Copy link
Member Author

bpasero commented Nov 8, 2016

@jtlowe the setting will not show up on Windows and Linux. And it will eventually be supported across all OS.

@smlombardi
Copy link

The app looks much nicer with this enabled, thanks for adding this.

@wprater
Copy link
Contributor

wprater commented Nov 8, 2016

Note that on macOS when "window.titleBarStyle": "custom" is set, the usual behavior allowing one to see the path tree (while holding down the command key and clicking) of the file in the titlebar is not preserved. I realize this is a "custom" titlebar, but it would still be nice to keep this behavior.

@smlombardi
Copy link

@wprater Perhaps an option to display the entire path in the title bar? I've seen that option in other editors.

@bpasero
Copy link
Member Author

bpasero commented Nov 9, 2016

@wprater good point, I guess one thing I could do is open a context menu upon Cmd+click that contains entries for the path segments to open in the finder. What I will not be able to do is to show icons though...

@bpasero
Copy link
Member Author

bpasero commented Nov 9, 2016

screen shot 2016-11-09 at 07 37 17

@bpasero
Copy link
Member Author

bpasero commented Nov 9, 2016

Btw we do also have a new option to show the full path in the title bar:

"window.showFullPath": true

@wprater
Copy link
Contributor

wprater commented Nov 9, 2016

I think users will appreciate that. I've used it many times over the years to open a directory from the title bar.

@bpasero
Copy link
Member Author

bpasero commented Nov 9, 2016

It should be in insiders now.

@bpasero
Copy link
Member Author

bpasero commented Nov 10, 2016

Closing, this is done minus bugs.

@bpasero bpasero closed this as completed Nov 10, 2016
@mikaoelitiana
Copy link
Contributor

Great job!! I am loving it

@wprater
Copy link
Contributor

wprater commented Nov 11, 2016

Closing, this is done minus bugs.

where is the bug list? thought I recalled you spelling them out? just wanted to track #15098 (comment)

@bpasero
Copy link
Member Author

bpasero commented Nov 11, 2016

@wprater but that one is fixed in current insider builds.

Outstanding bugs are two I can think of right now:

@egamma egamma mentioned this issue Dec 20, 2016
56 tasks
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality ux User experience issues
Projects
None yet
Development

No branches or pull requests

7 participants