Opened 5 years ago

Closed 5 years ago

#10940 closed bug (fixed)

ALT+OPT+CursorUp/Down doesn't close Tracker windows anymore

Reported by: humdinger Owned by: jscipione
Priority: normal Milestone: R1
Component: Applications/Tracker Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

This is hrev47363.

ALT+OPT+CursorUp/Down a Tracker window should close the window while opening the selected file/folder. It doesn't anymore. I feel that this is a quite recent regression, but can't say with which revision it definitely worked last...

Change History (6)

comment:1 by ttcoder, 5 years ago

My wild-ass-guess is that it occured circa hrev47362 ; if I understand correctly this changeset is in a system-wide keyboard "catcher" so it could affect (among others) Tracker shortcuts no ? Hopefully that can be fixed without reverting the change completely or dsuden is going to be disappointed 8-)

Last edited 5 years ago by ttcoder (previous) (diff)

comment:2 by jessicah, 5 years ago

The fix to Shortcuts shouldn't interfere when it has no handler for a particular keyboard combination.

Also, FWIW, I have this problem on hrev47342 as well. I also can't OPT-double-click on a folder to hide the parent window either.

comment:3 by ttcoder, 5 years ago

Gotta take that back.. I'm in hrev47358 here and the regression is already there.. So dunno when it happened (though I, too, remember using alt-opt-cursor successfully no long ago). I think the shortcut works on a friend's laptop that's running 47330 or whereabouts, will check to be sure.

Edit: Jessica beat me to it :-)

Last edited 5 years ago by ttcoder (previous) (diff)

comment:4 by jessicah, 5 years ago

Confirmed in hrev47342.

comment:5 by jscipione, 5 years ago

Owner: changed from axeld to jscipione
Status: newassigned

I'll take this one, probably broke it in recent Tracker changes.

comment:6 by jessicah, 5 years ago

Resolution: fixed
Status: assignedclosed

Fixed in hrev47373.

Note: See TracTickets for help on using tickets.