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: Show debug state in activity bar? #5841

Closed
bpasero opened this issue Apr 26, 2016 · 19 comments
Closed

Debug: Show debug state in activity bar? #5841

bpasero opened this issue Apr 26, 2016 · 19 comments
Assignees
Labels
debug Debug viewlet, configurations, breakpoints, adapter issues ux User experience issues
Milestone

Comments

@bpasero
Copy link
Member

bpasero commented Apr 26, 2016

We are not that aggressive anymore in opening the debug viewlet when you hit a breakpoint. Why not show the debug states as decoration on the debug viewlet itself? I could imagine a yellow pause icon when you are on a breakpoint and the green play icon when you are actually running or debugging a program.

@stevencl @chrisdias @bgashler1 thoughts?

@bpasero bpasero added debug Debug viewlet, configurations, breakpoints, adapter issues ux User experience issues labels Apr 26, 2016
@isidorn isidorn added this to the May 2016 milestone Apr 26, 2016
@isidorn
Copy link
Contributor

isidorn commented May 6, 2016

ping @bgashler1 @stevencl
silence I will take as approval and implement this on monday 😊

@stevencl
Copy link
Member

stevencl commented May 6, 2016

Sorry, take it as not keeping up :-)

In saying that, I am not opposed to the idea but I wonder why we changed the behaviour and don't switch to the debug viewlet when hitting a breakpoint now?

@bpasero
Copy link
Member Author

bpasero commented May 6, 2016

I am not sure if we should change the current behavior to always reveal the debug viewlet. I think we have a reason to not reveal it all the time because people got distracted. Thinking about viewlet indicators in general, I find it a good idea to show any viewlet state in the activity bar because these indicators are meant for exactly that purpose (the viewlet is not visible but has state that you might be interested in).

@isidorn
Copy link
Contributor

isidorn commented May 6, 2016

I like the arguments ben is making and that is the main reason why I would like to experiment with this to get feedback.

@bpasero
Copy link
Member Author

bpasero commented May 6, 2016

@stevencl or putting it the other way around, can you state concerns you have with showing the debug state in the viewlet? Would people maybe expect they could break/continue from the activity bar? Maybe we should start thinking of making the activity bar more powerful and allowing to invoke actions from it (e.g. the full debug toolbar).

@stevencl
Copy link
Member

stevencl commented May 6, 2016

I'm not opposed to the idea. If I had to think up a concern it would be the same one we have about icons in the explorer, that it becomes a bit of a Christmas tree effect. But that is just me trying to think of a concern. So definitely go ahead and experiment with it.

I see also that we switch to the debug viewlet until the user activates another viewlet during debug mode. That makes sense to me too.

@chrisdias
Copy link
Member

What are the badges that we would show in the activity bar? if we showed a play/pause badge it may feel like you can click on it to take an action. If it is multi color (e.g. orange when debugging) then i feel like it becomes a christmas tree very quickly.

@bpasero
Copy link
Member Author

bpasero commented May 6, 2016

Maybe @isidorn could attach screen shots if he has ideas how to present it. Also, I would maybe only show an indication when the execution is currently stopped in a breakpoint because otherwise we indicate debugging state in the status bar.

This would also better indicate that there is state in the debug viewlet to explore, because if the application is running, most of the views in the debug viewlet are empty anyways.

@bgashler1
Copy link
Contributor

image
Is this what you were thinking @isidorn ?

@chrisdias
Copy link
Member

@bgashler1 I want to click on that pause button!

@bgashler1
Copy link
Contributor

image

Since we already have all the Debug buttons persist on top (no matter what viewlet we're in), are we certain this is a problem with users?

  1. Are they often running their apps while working in the Explorer and wanting to be notified of hitting breakpoints?
  2. If so, is the "Pause" button turning to a "Play" button not clear enough that they have stopped due to a breakpoint/terminating?

@bgashler1
Copy link
Contributor

@chrisdias: just an experiment. Is this less clickable to you?
image

@chrisdias
Copy link
Member

No, its not :). One thing to consider, we have a problem where the debug toolbar only shows up when the debug viewlet is visible/active. i find this problematic if i switch to the explorer while at a break point, where did the toolbar go and why? we've always discussed having the activity bar buttons be commands, perhaps now is the time to consider.

@bpasero
Copy link
Member Author

bpasero commented May 7, 2016

Yes I was thinking the same when we brought up the concern that the icon looks like you could click it.

@isidorn
Copy link
Contributor

isidorn commented May 9, 2016

@bgashler1 yes I had something in mind like the first picture you attached.
@chrisdias I understand the concern that you might want to click on that pause button, but once the user clicks once and figures it is only a state he will no longer try to click it imho. It looks like the other badges which are also not clickable.

I am fine with pushing this to the backlog and introducing it once we decide we want to move forward on making activity bar buttons be commands

@stevencl
Copy link
Member

stevencl commented May 9, 2016

Incidentally, @chrisdias, I don't see the debug toolbar disappear when switching to the explorer viewlet while in debug mode. As @bgashler1 mentions above, the debug toolbar might be enough feedback to communicate the state of the debugger as long as it doesn't disappear when switching to another viewlet when in debug mode?

@isidorn
Copy link
Contributor

isidorn commented May 9, 2016

The debug toolbar does not dissapear

@chrisdias
Copy link
Member

I'm sorry, I was wrong about the toolbar disappearing when you switch viewlets. The thing that I was thinking about was that you have to bring up the debug viewlet to be able to press the "Play" button. When I move away from the debug viewlet, there is no button to press, and I like to press the button rather than hit F5 for some reason. I apologize for the confusion.

Going back to the original question... the problem is that we don't bring up the debug viewlet when we hit a breakpoint as aggressively as before. As brad says we have the toolbar in view and we have the orange status bar. Does the badge in the activity bar help us here?

@isidorn isidorn modified the milestones: Backlog, May 2016 May 11, 2016
@isidorn
Copy link
Contributor

isidorn commented Sep 5, 2016

Closing this as we have not reached a concenzus and I do not think adding this is solving any issue for the user.

@isidorn isidorn closed this as completed Sep 5, 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
debug Debug viewlet, configurations, breakpoints, adapter issues ux User experience issues
Projects
None yet
Development

No branches or pull requests

5 participants