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

Suggestion: Use different icons for Extension View popular and recommended buttons #8461

Closed
gregvanl opened this issue Jun 28, 2016 · 9 comments
Assignees
Labels
extensions Issues concerning extensions ux User experience issues verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@gregvanl
Copy link

  • VSCode Version:1.3.0-insiders
  • OS Version:Windows 7

The flame icon for popular is confusing with the OmniSharp flame and perhaps offensive ("hot"). The gift icon is difficult to see and understand.

Suggestion: Use a star icon for popular. This matches the star ratings we show in the extension items.

Suggestion: Use a thumbs up hand image for recommended. The thumbs up image is very popular on GitHub and easily understood.

@gregvanl gregvanl added the ux User experience issues label Jun 28, 2016
@bgashler1
Copy link
Contributor

Yes, just saw these and agree these could be improved. We might want to make this part of #8017

@bgashler1 bgashler1 self-assigned this Jun 28, 2016
@gregvanl
Copy link
Author

Can we use octicon-star and octicon-thumbsup? These are already available in the included octicon css.

@bgashler1
Copy link
Contributor

bgashler1 commented Jun 29, 2016

I emailed the icon team a request for new icons for this. Will update this issue when I hear back.

As placeholders in the interim, the star icon makes sense for popular, but thumbs-up does not. Thumbs up is often times associated with a direct action (+1 something, like something), so I wouldn't recommend it. Maybe you can use octicon-person temporarily for recommended.

In any case, we'll get some better icons for this soon.

@stevencl
Copy link
Member

stevencl commented Jun 29, 2016

We will add an icon for View - clicking on this we show a list of actions - show popular, show recommended, show installed, show updates.
Clicking these actions will populate the textbox with the appropriate command to perform the search.
Brad will get assets from the VS icon team to deliver to Isidor.

@bgashler1
Copy link
Contributor

bgashler1 commented Jun 29, 2016

Since all these commands are essentially filtering the list of all extensions by criteria we provide (such as whether it's installed, or whether it's popular or recommended), the icon team and I felt that a "Filter" icon would be appropriate.

I've attached the asset here.
Filter_16x.zip

screen-shot-2016-06-29-at-10 59 43-am

@bgashler1 bgashler1 assigned joaomoreno and unassigned bgashler1 Jun 29, 2016
@gregvanl
Copy link
Author

Given that some of the actions are filtering (@popular, @recommended) and some are local (installed, installed and outdated), I think a filter icon is confusing. I'd prefer "..." which is clear to mean a dropdown will appear.

@bgashler1
Copy link
Contributor

@gregvanl They're actually still filtered from all available extensions. If you were to scroll to each point in the entire list of extensions where you have installed an extension, you will see the same stuff.

@isidorn isidorn assigned isidorn and unassigned joaomoreno Jun 30, 2016
@isidorn isidorn reopened this Jun 30, 2016
@isidorn isidorn added the extensions Issues concerning extensions label Jun 30, 2016
@isidorn isidorn added the verification-needed Verification of issue is requested label Jun 30, 2016
@isidorn isidorn assigned bpasero and unassigned isidorn Jun 30, 2016
@isidorn
Copy link
Contributor

isidorn commented Jul 1, 2016

After getting negative feedback for the filter icon we have chosen to go with the elipses. Our concern with the elipses is that it goes against our design principle if it is the only action. So we also moved the clear action top level to also address #8386.
We felt this solution is a nice compromise between various suggestions.

screen shot 2016-07-01 at 12 23 08

@bpasero
Copy link
Member

bpasero commented Jul 1, 2016

@isidorn make sure to fix the bad outline you now get that you removed the search attribute from input.

isidorn added a commit that referenced this issue Jul 1, 2016
@chrmarti chrmarti added the verified Verification succeeded label Jul 1, 2016
@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
extensions Issues concerning extensions ux User experience issues verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

8 participants