Edgewall Software
Modify

Ticket #7867 (new enhancement)

Opened 3 years ago

Last modified 2 years ago

h,l keyboard shortcuts

Reported by: techtonik <techtonik@…> Owned by:
Priority: normal Milestone: next-major-0.1X
Component: version control/browser Version: none
Severity: normal Keywords:
Cc:
Release Notes:
API Changes:

Description

A request to add h and l to move up and down (or better say left and right) the directory tree structure expanding/collapsing leaves as needed.

There is also a bug in j,k shortcuts that they stop work after <Tab> is pressed.

Attachments

Change History

comment:1 Changed 3 years ago by cboos

Interesting idea, but there are already two assigned keys that can be used to expand/collapse a directory ('o' and 'Enter'), so I'm not sure adding 'l' would be of great help. Also, in attachment:7074-directory-expand-and-nav.2.patch:ticket:7074 I use 'l' as a shortcut for getting to the 'l'og view.

comment:2 Changed 3 years ago by techtonik <techtonik@…>

I guess existing shortcuts were made after Google Experimental Search, but Google uses them to navigate plain top-down list and this is insufficient for convenient tree browsing.

Move up and down the tree actions are more sophisticated than simple expand/collapse.
Move down the tree (l key or "right"):

  1. if folder is selected
    1.1 if folder is not expanded - expand it
    1.2 if the folder is expanded - move selection to the first child
  2. if there is file under cursor - do nothing

Move up the tree (h key - "left"):

  1. if expanded folder is selected - collapse the folder
  2. if file or collapsed folder is selected - move to parent node

As for existing 'l'og shortcut - it is convenient, but may be too easy to hit 'l' when navigating sources. Can it be replaced by other key, to say 'r' for revision log?

comment:3 Changed 3 years ago by cboos

  • Milestone set to 0.12

I'll have a second look on this after committing my pending changes to the keyboard navigation.

comment:4 Changed 2 years ago by rblank

  • Milestone changed from 0.12 to next-minor-0.12.x

Should probably not block 0.12.

comment:5 Changed 2 years ago by cboos

  • Milestone changed from next-minor-0.12.x to next-major-0.1X

I'll change the current 'l'og shortcut to 'L'og, so this leaves room for implementing this suggestion in a later version.

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as new
as The resolution will be set. Next status will be 'closed'
to The owner will be changed from (none). Next status will be 'new'
The owner will be changed from (none) to anonymous. Next status will be 'assigned'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.