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

Default sidebar width should not trigger narrow mode for extensionsViewlet #118511

Closed
Tracked by #126479
digitarald opened this issue Mar 9, 2021 · 12 comments
Closed
Tracked by #126479
Assignees
Labels
extensions Issues concerning extensions feature-request Request for new features or functionality getting-started on-testplan ux User experience issues
Milestone

Comments

@digitarald
Copy link
Contributor

digitarald commented Mar 9, 2021

The default sidebar width is dynamically calculated but ends up being 251px for most window sizes I tested. The narrow mode for the extensionsViewlet is triggered for ⪬ 300px.

While narrow is great for small spaces, it hides key information like the icon, rating and download numbers. For users who don't discover the resizing, important context is lost for extensions. I noticed this during my work on getting started, as it guides users early into the extension panel; which looks no very inviting without expanded.

Ideas for solutions

  • Trigger narrow for ⪬ 250px
  • Add a size before narrow that allows more context like a smaller extension icon and simpler rating, or other redesign ideas.

Tooltips on descriptions to read cut off text.

cc @sana-ajani @misolori @fiveisprime

@digitarald digitarald added bug Issue identified by VS Code Team member as probable bug getting-started labels Mar 9, 2021
@gjsjohnmurray
Copy link
Contributor

@digitarald you are referring to the side bar, not the activity bar. See https://code.visualstudio.com/docs/getstarted/userinterface#_basic-layout and the annotated screenshot above that section.

Please update the issue.

@sana-ajani
Copy link
Contributor

Good observation! Strongly agree that that showing icons, the starred recommended ones, and download count would provide a richer view and catch new users' eyes.

@digitarald digitarald changed the title Default activity bar width should not trigger narrow mode for extensionsViewlet Default sidebar width should not trigger narrow mode for extensionsViewlet Mar 9, 2021
@miguelsolorio
Copy link
Contributor

I think your proposal is to change the default width of the sidebar (not just for the extensions view). I can't remember if @sbatten or @isidorn owns the sidebar?

@isidorn
Copy link
Contributor

isidorn commented Mar 9, 2021

Please note that the SideBar probably takes the "optimal" width of the explorer since that is the initial view shown.
When we change the active view in the sidebar the sidebar does not change size. Not sure if that would be strange.

Anyways leaving this up to @sbatten to decide

@digitarald
Copy link
Contributor Author

I think your proposal is to change the default width of the sidebar (not just for the extensions view).

I am leaning towards the lower hanging proposal to change the min-width for narrow. For changing the default sidebar width I'd have to dig deeper into seen window sizes. I am also not aware that other panels have problems with narrow width; which would otherwise strengthen the case. The middle-ground would be increasing sidebar default width a bit and making the narrow min-width a bit wider.

@sbatten
Copy link
Member

sbatten commented Mar 9, 2021

@sandy081 for the narrow breakpoint in extensions viewlet

@miguelsolorio
Copy link
Contributor

Ah, ok I see what you mean now. That makes sense to me as well, this is handled here:

layout(dimension: Dimension): void {
if (this.root) {
this.root.classList.toggle('narrow', dimension.width <= 300);
}
if (this.searchBox) {
this.searchBox.layout(new Dimension(dimension.width - 34, 20));
}
super.layout(new Dimension(dimension.width, dimension.height - 41));
}

@digitarald
Copy link
Contributor Author

digitarald commented Mar 9, 2021

@sandy081 sandy081 added the extensions Issues concerning extensions label Mar 11, 2021
@sandy081 sandy081 added this to the March 2021 milestone Mar 11, 2021
@sandy081 sandy081 added the ux User experience issues label Mar 11, 2021
@sandy081
Copy link
Member

It seems showing icon, title, version, rating & downloads in 250 px is too clumsy - title and version are shown partially with ellipsis.

What do you think not showing version instead of rating & downloads in narrow version?

@miguelsolorio
Copy link
Contributor

@sandy081 I think when you are installing an extension for the first time you are less concerned about the version so that sounds good to me. Screenshot?

@digitarald
Copy link
Contributor Author

Download numbers seem the best indicator (and are also used in Popular), as ratings can mislead.

If the entries use a grid layout, it could shift the icon from top-row to the left:

image

@sandy081 sandy081 modified the milestones: March 2021, April 2021 Mar 24, 2021
@sandy081 sandy081 modified the milestones: April 2021, May 2021 Apr 26, 2021
@sandy081 sandy081 modified the milestones: May 2021, June 2021 Jun 3, 2021
@sandy081 sandy081 added feature-request Request for new features or functionality and removed bug Issue identified by VS Code Team member as probable bug labels Jun 30, 2021
@sandy081 sandy081 removed this from the June 2021 milestone Jun 30, 2021
@sandy081
Copy link
Member

Improved the viewlet representation when the width is small. It also has other improvements which are part of #126479. Feedback is appreciated

Kapture.2021-07-15.at.15.33.51.mp4

@github-actions github-actions bot locked and limited conversation to collaborators Sep 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
extensions Issues concerning extensions feature-request Request for new features or functionality getting-started on-testplan ux User experience issues
Projects
None yet
Development

No branches or pull requests

7 participants