View Issue Details

IDProjectCategoryView StatusLast Update
0001968ardourbugspublic2008-11-21 00:03
Reportercolinf Assigned Topaul  
PrioritynormalSeveritytrivialReproducibilityalways
Status closedResolutionfixed 
Product VersionSVN/2.0-ongoing 
Summary0001968: "zoom in" & "zoom out" default key bindings are (still) reversed (IMHO)
DescriptionI think the default key bindings for "zoom in" ('-') and "zoom out" ('=', i.e. the '+' key) are the wrong way round, especially since the icon on the "zoom in" button is a '+' and on the "zoom out" button a '-'.

Obviously it's easy to set these to whatever one likes with the key binding editor, or by editing ~/.ardour2/ardour.bindings, but I don't see why it shouldn't be consistent from the outset.

I think the bindings have always been the current way around, so maybe there's a reason for it, but in case anybody else agrees with me, I've attached an (entirely trivial) patch to swap the two keys over.
TagsNo tags attached.

Activities

2007-11-22 13:35

 

swap-zoom-in-out-keys.patch (1,213 bytes)   
Index: gtk2_ardour/ardour.bindings.in
===================================================================
--- gtk2_ardour/ardour.bindings.in	(revision 2710)
+++ gtk2_ardour/ardour.bindings.in	(working copy)
@@ -202,7 +202,7 @@
 ; (gtk_accel_path "<Actions>/Main/ExportSession" "")
 ; (gtk_accel_path "<Actions>/options/InputAutoConnectPhysical" "")
 ; (gtk_accel_path "<Actions>/Snap/snap-to-edit-cursor" "")
-(gtk_accel_path "<Actions>/Editor/temporal-zoom-in" "minus")
+(gtk_accel_path "<Actions>/Editor/temporal-zoom-in" "equal")
 ; (gtk_accel_path "<Actions>/JACK/Latency" "")
 (gtk_accel_path "<Actions>/Editor/edit-cursor-to-range-end" "<Shift>F2")
 ; (gtk_accel_path "<Actions>/redirectmenu/rename" "")
@@ -301,7 +301,7 @@
 ; (gtk_accel_path "<Actions>/redirectmenu/selectall" "")
 (gtk_accel_path "<Actions>/Editor/editor-copy" "<Control>c")
 ; (gtk_accel_path "<Actions>/Snap/snap-to-quarters" "")
-(gtk_accel_path "<Actions>/Editor/temporal-zoom-out" "equal")
+(gtk_accel_path "<Actions>/Editor/temporal-zoom-out" "minus")
 ; (gtk_accel_path "<Actions>/options/UseSoftwareMonitoring" "")
 ; (gtk_accel_path "<Actions>/Editor/Subframes100" "")
 (gtk_accel_path "<Actions>/Editor/mute-unmute-region" "m")
swap-zoom-in-out-keys.patch (1,213 bytes)   

paul

2008-04-21 13:11

administrator   ~0004887

fixed (for some time)

Issue History

Date Modified Username Field Change
2007-11-22 13:35 colinf New Issue
2007-11-22 13:35 colinf File Added: swap-zoom-in-out-keys.patch
2008-04-21 13:11 paul Status new => resolved
2008-04-21 13:11 paul Resolution open => fixed
2008-04-21 13:11 paul Assigned To => paul
2008-04-21 13:11 paul Note Added: 0004887
2008-11-21 00:03 seablade Status resolved => closed