RFR: JDK-8288750: IGV: Improve Shortcuts [v7]
Tobias Holenstein
tholenstein at openjdk.org
Mon Jun 27 08:02:04 UTC 2022
> *Improvement of keyboard shortcuts in IGV under macOS:*.
> Certain keyboard/mouse shortcuts do not work under macOS. E.g. `Ctrl + left-click` to select multiple nodes. The reason is that this keyboard shortcut is hardwired as a right-click under macOS and cannot be easily changed in the operating system. In general, the macOS user manual recommends using "Command/Meta" as a modifier key instead of "Control."
>
> *Fixed focus of the Graph Tab:*.
> In IGV, shortcuts are linked to a component. Components are for example a Graph Tab, "Outline", "Filters", "Bytecode", "Control Flow" and "Properties". Shortcuts only work if the linked component is in focus. The focus can be changed with `Ctrl + TAB` or by clicking into the TAB component. The Graph Tab did not get the focus back when the user clicked on it. This needed to be fixed.
>
> *Fixing QuickSearch:*
> Netbeans' QuickSearchAction is a global component of which only one common instance exists. IGV used a workaround to repaint the search bar in a new graphics tab. On macOS, the search bar doubled in size with each new Graph Tab. In addition, keyboard shortcuts for the search bar did not work. This issue was fixed by adding the search bar whenever the tab gained focus, and removing it (by default) when a new tab gained focus. This way, no workaround is required, and the size and ability to use a keyboard shortcut are fixed.
>
> *Adding new actions to expand/shrink the difference selection:*.
> The user can expand/reduce the difference selection by moving the beginning/end of the selection with the mouse.
> 
> This is something many users didn't know. Therefore two new buttons should make it more clear for the user that this functionality exists.
> 
> By adding these button we can now also add keyboard shortcuts to expand/reduce the difference selection.
>
> **Fixed shortcuts for:**
> - Add a single node in the graph to selection (`Ctrl/Cmd + left-click`)
> - Add a multiple node in the graph to selection (`Ctrl/Cmd + left-click-drag`)
> - Zoom in and out (`Ctrl/Cmd + mouse-wheel`)
>
> **Added new shortcuts for:**
> - Search (`Ctrl/Cmd - I` and `Ctrl/Cmd - F`)
> - Undo (`Ctrl/Cmd - Z`)
> - Redo (`Ctrl/Cmd - Y` and `Ctrl/Cmd - Shift - Z`)
> - Show Next Graph (`Ctrl/Cmd - RIGHT`)
> - Expand the difference selection (`Ctrl/Cmd - UP` and `Ctrl/Cmd - Shift - RIGHT`)
> - Reduce the difference selection (`Ctrl/Cmd - DOWN` and `Ctrl/Cmd - Shift - LEFT`)
> - Show Previous Graph (`Ctrl/Cmd - LEFT`)
> - Show satellite view (`Hold S`)
Tobias Holenstein has updated the pull request incrementally with one additional commit since the last revision:
code style
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/9260/files
- new: https://git.openjdk.org/jdk/pull/9260/files/c9dbe843..fed782ad
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=9260&range=06
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=9260&range=05-06
Stats: 29 lines in 6 files changed: 0 ins; 20 del; 9 mod
Patch: https://git.openjdk.org/jdk/pull/9260.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/9260/head:pull/9260
PR: https://git.openjdk.org/jdk/pull/9260
More information about the hotspot-compiler-dev
mailing list