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

Debug: Icon issues in call stack view #84573

Closed
joaomoreno opened this issue Nov 12, 2019 · 6 comments
Closed

Debug: Icon issues in call stack view #84573

joaomoreno opened this issue Nov 12, 2019 · 6 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Milestone

Comments

@joaomoreno
Copy link
Member

joaomoreno commented Nov 12, 2019

The disconnect icon is missing:

image

Vertical alignment is broken:

image

Icon visibility is broken in a selected yet not focused row (click row and press Down):

image

Icon visibility for disabled actions isn't great for selected and focused rows:

image

Weird mouse hover behavior:

Peek 2019-11-12 11-18

@joaomoreno joaomoreno changed the title Debug: Disconnect icon is missing from call stack view Debug: Icon issues in call stack view Nov 12, 2019
@isidorn isidorn added bug Issue identified by VS Code Team member as probable bug debug Debug viewlet, configurations, breakpoints, adapter issues labels Nov 12, 2019
@isidorn isidorn added this to the November 2019 milestone Nov 12, 2019
@isidorn
Copy link
Contributor

isidorn commented Nov 12, 2019

@joaomoreno nice catches. I have pushed a fix which tackles everything except the "Disabled action visibility is not great for selected items". That one I leave up to @misolori if he would like to tackle.

Also note that the vertical alignment I got with the vertical-align: text-top. I am open for suggestions on a nicer solution, though this seems to work fine.

Let me know how it behaves now.

@miguelsolorio
Copy link
Contributor

I think that it should still look disabled in the active/selected state, I'll take that one.

Also concerned that the stop icon looks like this:

image

Will look into this as well.

@miguelsolorio
Copy link
Contributor

Actually, the disabled opacity is set to 40% across all disabled items in the product, which I don't think we should change:

.monaco-action-bar .action-item.disabled .action-label,
.monaco-action-bar .action-item.disabled .action-label:hover {
opacity: 0.4;
}

@isidorn
Copy link
Contributor

isidorn commented Nov 14, 2019

@misolori if you are refering to the centering regardign the stop icon: I should have fixed that.

@miguelsolorio
Copy link
Contributor

@isidorn no, I meant how the stop icon is filled and not an outline. It should be like this (I already fixed this):

image

@isidorn
Copy link
Contributor

isidorn commented Nov 14, 2019

Yes, that makes senst. Thanks!

@roblourens roblourens added the verified Verification succeeded label Dec 5, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Dec 27, 2019
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 debug Debug viewlet, configurations, breakpoints, adapter issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants