Skip to content

Comments

style(focus): avoid two spaces with "Focus on {view} View" when no view name is specified#296873

Open
kilianpaquier wants to merge 1 commit intomicrosoft:mainfrom
kilianpaquier:fix/focus-on-spaces
Open

style(focus): avoid two spaces with "Focus on {view} View" when no view name is specified#296873
kilianpaquier wants to merge 1 commit intomicrosoft:mainfrom
kilianpaquier:fix/focus-on-spaces

Conversation

@kilianpaquier
Copy link

Hi Team 👋

A small pull request to not have two spaces when a viewDescriptor doesn't have a name. Might happen with extensions when the view "namespace" itself is self-sufficient (and to avoid duplication):

<Namespace>: Focus on <Namespace> View.

Some examples (it's not bad but some extensions might choose to avoid this duplication):

Ports: Focus on Ports View.
Chat: Focus on Chat View.
Output: Focus on Output View.
Problems: Focus on Problems View.

…ew name is specified

Signed-off-by: kilianpaquier <git@kilianpaquier.dev>
@vs-code-engineering
Copy link

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@sandy081

Matched files:

  • src/vs/workbench/services/views/browser/viewsService.ts

@benibenj

Matched files:

  • src/vs/workbench/services/views/browser/viewsService.ts

@bpasero

Matched files:

  • src/vs/workbench/services/views/browser/viewsService.ts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants