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

Quickpick items search reveals octicons used in the items #21243

Closed
eamodio opened this issue Feb 23, 2017 · 13 comments
Closed

Quickpick items search reveals octicons used in the items #21243

eamodio opened this issue Feb 23, 2017 · 13 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug quick-pick Quick-pick widget issues verified Verification succeeded
Milestone

Comments

@eamodio
Copy link
Contributor

eamodio commented Feb 23, 2017

  • VSCode Version: 1.10.0-insider 256d76f
  • OS Version: macOS Sierra 10.12.3 (16D32)

Steps to Reproduce:

  1. Open a quick pick menu that uses the built in octicons in the items
    quickpick-before

  2. Type a character that is in the octicon name
    quickpick-after

  3. Octicons get revealed as text

@roblourens roblourens added the bug Issue identified by VS Code Team member as probable bug label Feb 23, 2017
@bpasero bpasero modified the milestones: Backlog, February 2017 Feb 23, 2017
@eamodio
Copy link
Contributor Author

eamodio commented Mar 2, 2017

It also looks like the octicon text also shows up in the hover tooltips (title attr)

@eamodio
Copy link
Contributor Author

eamodio commented Jun 8, 2017

Any chance to get this on the list for the June (or soon) iteration? This does seem to cause confusion for users as they have no idea what that text that all of a sudden appears is.

@jens1o
Copy link
Contributor

jens1o commented Jun 8, 2017

Maybe a code position where we could dig in?

@bpasero
Copy link
Member

bpasero commented Jun 10, 2017

The picker is opened from here: https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/browser/parts/quickopen/quickOpenController.ts#L219

@jens1o
Copy link
Contributor

jens1o commented Jun 10, 2017

Thanks, I'll take a look, but I was never that deep in the code. :/

@bpasero bpasero added quick-pick Quick-pick widget issues and removed workbench labels Nov 15, 2017
@bpasero bpasero removed this from the Backlog milestone Nov 15, 2017
@bpasero bpasero added this to the February 2018 milestone Feb 7, 2018
@bpasero bpasero closed this as completed in ee9f1e7 Feb 7, 2018
@bpasero
Copy link
Member

bpasero commented Feb 7, 2018

@eamodio maybe you check the fix out on master with GitLens since you are a heavy customer.

@eamodio
Copy link
Contributor Author

eamodio commented Feb 7, 2018

Will do - thanks!

@eamodio
Copy link
Contributor Author

eamodio commented Feb 7, 2018

@bpasero I just checked out master and I don't see any tooltips on quickpick items now.

I did notice another issue (not new though), the tooltip over the input (for the placeholder), shows the wrong content (previous content) if the quickpick is replaced with a new one.

For example, in GitLens if you open file or branch history in a quickpick and see the tooltip over the input (it should be correct), but then pick one of the commits (which will open a new quickpick), and see that the tooltip over the input it still showing the previous tooltip content, not the updated content. (FYI, this is not new, I can reproduce it in 1.19 too)

@bpasero
Copy link
Member

bpasero commented Feb 7, 2018

@eamodio can you file separate issues for that? To be clear: this issue is resolved for you right?

@eamodio
Copy link
Contributor Author

eamodio commented Feb 7, 2018

@bpasero No -- now I don't see any tooltips at all -- I was expecting that the octicons would be stripped, but now there is no tooltip at all.

Will do, re: other issue.

@eamodio
Copy link
Contributor Author

eamodio commented Feb 7, 2018

Opened: #43114 for the other issue

@eamodio
Copy link
Contributor Author

eamodio commented Feb 7, 2018

Sorry, I was getting mixed up on this. The search part works great now -- octicons don't show up

@jrieken jrieken added the verified Verification succeeded label Feb 28, 2018
@jrieken
Copy link
Member

jrieken commented Feb 28, 2018

adding verified label based on @eamodio's comment

@vscodebot vscodebot bot locked and limited conversation to collaborators Mar 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug quick-pick Quick-pick widget issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants