Change Fullscreen Hotkey on OSX

Bug #615628 reported by Konrad Windszus
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
jus
1.8
Fix Released
Low
jus

Bug Description

I would strongly suggest to change the default hotkey for fullscreen to something different than F11, since that key is by default already bound to Expose in Mac OS X. All F-Keys from F7 to F12 are actually bound by default, so I recommend using another Hotkey. What about Shift+Cmd/Ctrl+F like in Firefox?

Related branches

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Hi Konrad,

Thanks for the suggestion. F11 is the 'standard' key in both GNOME and Windows environments, so that was the reasoning behind the switch. If this is not the case in OSX, then it makes sense that we should consider changing it to be the community standard for OSX.

We could just special case per OS for what the hotkey is.

Thanks,
RJ

Changed in mixxx:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
jus (jus) wrote :

Good point.

Actually F11 key is assigned differently on OSX depending on OS version / Hardware. I.e on macbook pro OSX 10.6 "F11" is "Lower volume".
There is no standard key for "Fullscreen" see http://support.apple.com/kb/HT1343.

I would go go for the simplest shortcut "CMD+F". Although this shortcut is assigned with inline searching for text finder/editors/browsers, it is quasi standard for fullscreen too. As seen on Apple Quicktime / DVD Player, VLC, Mixvibes Cross.
Also we already have "CMD+Y" for vinyl control, "CMD+O" for load song etc....

Even though" Shift+Cmd+F" would be a good alternative.

summary: - Change Fullscren Hotkey
+ Change Fullscreen Hotkey
summary: - Change Fullscreen Hotkey
+ Change Fullscreen Hotkey on OSX
Revision history for this message
jus (jus) wrote :

Patch changes the fullscreen hotkey to "CMD+F" for OSX.
Tested and working with lp:mixxx/1.8

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Thanks Jus, applied to lp:mixxx and lp:mixxx/1.8

Changed in mixxx:
status: Confirmed → Fix Committed
assignee: nobody → jus (jus)
RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/5473

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.