Opened 8 years ago

Closed 6 years ago

#11979 closed bug (fixed)

Missing shortcuts in file panels

Reported by: humdinger Owned by: waddlesplash
Priority: normal Milestone: R1
Component: Kits/ Version: R1/Development
Keywords: Cc: jessicah
Blocked By: Blocking: #12096
Platform: All


This is hrev49043.

There are some keyboard shortcuts missing from the open/save file panels.

  • ALT+Up doesn't send you up one level to the parent folder.
  • ALT+W should close the panel.

I also happened upon some panels where ALT+H and ALT+D don't send you to Home and Desktop. It works on some apps and doesn't on others. StyledEdit is such a case where it doesn't work. In Pe on the other, it works...

Attachments (2)

Change History (27)

comment:1 by waddlesplash, 8 years ago

Owner: changed from nobody to waddlesplash
Status: newassigned

comment:2 by diver, 8 years ago

Blocking: 12096 added

comment:3 by diver, 8 years ago

Milestone: UnscheduledR1/beta1

comment:4 by jessicah, 8 years ago

Cc: jessicah added

comment:5 by pulkomandy, 8 years ago

Milestone: R1/beta1R1

comment:6 by diver, 7 years ago

Maybe we could create a GCI task to do a git bisect to find out when shortcuts stopped working?

comment:7 by waddlesplash, 7 years ago

It probably was the tracker_layout commit, but as I think I said before, I've no clue how or why :(

If someone who knows more about Tracker and BView internals could look at this, that'd be best...

comment:8 by diver, 6 years ago

It looks like jessicah tried to fix it in 89486c92764891b1142243547d8e97fdc866c4f1 but then reverted it in hrev49212.

The bug is pretty annoying, could someone please take a look at it again?

comment:9 by owenca, 6 years ago

Both Alt+H and Alt+D keys work in Pe, StyleEdit, Expander, and Filer, so I only have to make Alt+Up and Alt+W work. See the attached patch below.

comment:10 by owenca, 6 years ago

patch: 01

comment:11 by owenca, 6 years ago

Please disregard or remove the above patch. I will update the patch for #13151, and it will make Alt-Up work, too.

comment:12 by pulkomandy, 6 years ago

patch: 10

comment:13 by owenca, 6 years ago

I have a new patch, which fixes this ticket (and this ticket only).

Last edited 6 years ago by owenca (previous) (diff)

comment:14 by owenca, 6 years ago

patch: 01

comment:15 by diver, 6 years ago

Does Alt+Left/Alt+Right work as well (after navigation to some directory)?

in reply to:  15 comment:16 by owenca, 6 years ago

I tested the patch with Tracker, Expander, and StyleEdit and saw no change in the behavior of Alt+Left/Right keys.

Last edited 6 years ago by owenca (previous) (diff)

comment:17 by diver, 6 years ago

I meant in filepanel itself.

comment:18 by owenca, 6 years ago

Yes, I tested it in the file panels of Expander and StyleEdit. I didn't see any difference. Neither Alt-Left nor Alt-Right did anything before and after the patch was applied. What are they supposed to do?

comment:19 by diver, 6 years ago

Just like in Tracker they're supposed to navigate to previous/next folder.

comment:20 by owenca, 6 years ago

That's why I said I compared the behavior of Alt-Left/Alt-Right across Tracker, Expander, and StyleEdit. They didn't have any effect with or without the patch. Left/Right works though.

comment:21 by owenca, 6 years ago

After you showed me how to set Tracker browser to single window, I can see what Alt-Left and Alt-Right keys do. :)

comment:22 by humdinger, 6 years ago

I have checked out the file panel behaviour with hrev51347 - with and without Owen's patches of #11979 and #13151.

Without patches:

App Panel Alt+H |Alt+D Alt+Up Alt+W Esc-close Esc-reset-filtering
ExpanderOpen**--* ** *
FilerOpen**---* *

With #11979 and #13151:

App Panel Alt+H |Alt+D Alt+Up Alt+W Esc-close Esc-reset-filtering
PeOpenokokokokok* * *
Saveokokokokok* * *
StyledEditOpen**okokok* * *
Saveokokokokok* * *
ShowImageOpen**okokok* * *
Saveokokokokok* * *
MediaPlayerOpenokokokokok* * *
(playlist)Saveokokokokok* * *
ExpanderOpen**okokok* * *
FilerOpen**okokok* * *

* works after clicking into the file list or moving the selection with the cursor keys (even though file list had the blue focus border already)

* * Esc hides all files of the list

* * * Closes the panel instead of resetting the type-ahead-filtering

So, with the patches there's quite an improvement. I'd appreciate if someone more familiar with the Tracker code could look at the code changes and apply the patch.

The focus issues marked with * are a bit puzzling...
What should be improved is the ESC behaviour when the type-ahead filter is active. Then, the ESC should only reset/deactivate the filter. Currently that closes the panel right away (* * *).
See also #13151.

comment:23 by pulkomandy, 6 years ago

So we should fix at least handling of the ESC key before merging the patches. The issue with * being already present in some cases, we can investigate it after merging these patches.

comment:24 by owenca, 6 years ago

The patch attached to this ticket only fixes Alt+Up and Alt+W and should not change the behavior of Esc. Maybe we can merge this patch first?

comment:25 by waddlesplash, 6 years ago

Resolution: fixed
Status: assignedclosed

Applied (this ticket's patch only) in hrev51542. Thanks!

Note: See TracTickets for help on using tickets.