-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
Comments
ping @bgashler1 @stevencl |
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? |
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). |
I like the arguments ben is making and that is the main reason why I would like to experiment with this to get feedback. |
@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). |
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. |
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. |
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 I want to click on that pause button! |
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?
|
@chrisdias: just an experiment. Is this less clickable to you? |
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. |
Yes I was thinking the same when we brought up the concern that the icon looks like you could click it. |
@bgashler1 yes I had something in mind like the first picture you attached. 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 |
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? |
The debug toolbar does not dissapear |
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? |
Closing this as we have not reached a concenzus and I do not think adding this is solving any issue for the user. |
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?
The text was updated successfully, but these errors were encountered: