Bug 404406 - If there is a selection, the 'step into' should run 'step into selection'
Summary: If there is a selection, the 'step into' should run 'step into selection'
Status: ASSIGNED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 4.3   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: JDT-Debug-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-26 15:55 EDT by Alex Blewitt CLA
Modified: 2016-09-19 13:40 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Blewitt CLA 2013-03-26 15:55:16 EDT
When the 'step into' menu bar action is selected, if there is a selection it should invoke 'step into selection' if the selection is in the same method as the executing context. This may be better than having an additional icon in the menu bar for 'step into selection'. Alternatively, alt-clicking on the 'step into' might perform a 'step into selection' instead.
Comment 1 Dani Megert CLA 2013-03-27 05:42:35 EDT
(In reply to comment #0)
> When the 'step into' menu bar action is selected, if there is a selection it
> should invoke 'step into selection' if the selection is in the same method
> as the executing context. 

I like that idea!


> This may be better than having an additional icon
> in the menu bar for 'step into selection'. 

Yes. I would not want to add more clutter to it.


> Alternatively, alt-clicking on
> the 'step into' might perform a 'step into selection' instead.

Not sure what you wanted to say here. Ctrl+Alt click does exactly that.
Comment 2 Alex Blewitt CLA 2013-03-27 07:52:16 EDT
(In reply to comment #1)
> (In reply to comment #0)
> > Alternatively, alt-clicking on
> > the 'step into' might perform a 'step into selection' instead.
> 
> Not sure what you wanted to say here. Ctrl+Alt click does exactly that.

Ctrl+Alt on the text in the source does 'step into selection'. I was thinking about people using the 'step into' button on the toolbar; or does Ctrl+Alt on that toolbar button do a 'step into'?

Having said that, if the selection present makes the 'step into' button do that anyway, the way that you click on the toolbar button becomes irrelevant anyway.
Comment 3 Markus Keller CLA 2013-03-27 08:11:06 EDT
The Step Into and Step Into Selection commands must stay predictable.

-1 for a magic change of behavior of "Step Into". I have "Step Into Selection" bound to F9, and I use both F5 and F9 depending on what I need. Taking away "Step Into" in some situations is a no-go.

Adding a modifier for the toolbar button to change it to "Step Into Selection" would be fine, but it should be Ctrl (Mac: Command) for consistency with other action modifiers and because Alt+drag moves windows on Linux.
Comment 4 Dani Megert CLA 2013-03-27 12:38:19 EDT
(In reply to comment #3)
> The Step Into and Step Into Selection commands must stay predictable.
> 
> -1 for a magic change of behavior of "Step Into". I have "Step Into
> Selection" bound to F9, and I use both F5 and F9 depending on what I need.
> Taking away "Step Into" in some situations is a no-go.

Indeed.


I'm not sure the modifier buys us something. I would expect that those who use the mouse (to click the button), would better use Alt+Ctrl+click then.
Comment 5 Michael Rennie CLA 2013-04-01 12:31:47 EDT
(In reply to comment #3)
> The Step Into and Step Into Selection commands must stay predictable.
> 
> -1 for a magic change of behavior of "Step Into". I have "Step Into
> Selection" bound to F9, and I use both F5 and F9 depending on what I need.
> Taking away "Step Into" in some situations is a no-go.
> 

+1. Some of the debug commands are confusing enough, I don't want any more of them to behave differently based on workbench selections (see context launching).

> Adding a modifier for the toolbar button to change it to "Step Into
> Selection" would be fine, but it should be Ctrl (Mac: Command) for
> consistency with other action modifiers and because Alt+drag moves windows
> on Linux.

We already do this for the launch toolbar: Ctrl + Click will create a new launch configuration (using context launching logic) and open the launch dialog on it - or just open the launch dialog if a config could not be created.
Comment 6 Michael Rennie CLA 2013-04-01 12:43:21 EDT
Removing milestone. I likely won't have time to do this for M7 as I just remembered that the entirety of the Step Into action code is in platform debug and the entirety of the Step Into Selection code is in JDT debug (as resolving the selection requires the Java model).