Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add keyboard shortcut to maximize library view #7892

Closed
mixxxbot opened this issue Aug 22, 2022 · 11 comments
Closed

Add keyboard shortcut to maximize library view #7892

mixxxbot opened this issue Aug 22, 2022 · 11 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: esbrandt
Date: 2015-03-11T15:14:06Z
Status: Fix Released
Importance: Low
Launchpad Issue: lp1430847
Tags: keyboard, skin, usability


Often it is useful to have a bigger library view, and grabbing the mouse/trackpad to press a button not an option.
I propose to use the Space bar to toggle the max library view.

Deliverables:

  • Find naming convention for control. Currently skins use either [Master],show_library or [Library],unshow_big.
    I propose to use [Master],maximize_library
  • Add shortcut for control to all *.kbd.cfg
  • Update existing tooltip wording
@mixxxbot
Copy link
Collaborator Author

Commented by: esbrandt
Date: 2015-03-11T19:16:07Z


#519

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2015-03-12T13:00:38Z


Many controllers have encoders or buttons for navigating the library. For those that have mappings already, it would be nice if they were updated to take advantage of this new control.

@mixxxbot
Copy link
Collaborator Author

Commented by: rafvdm
Date: 2015-05-06T15:26:38Z


I noticed with the current 1.12-r4518 beta-build, that the space bar maximizes the library view.
So I guess this option is now introduced.

But please note, that I have "shortcut keys DISABLED" and the space bar still does this trick!

Unless I'm wrong, with shortcut keys disabled this should not be functioning.

@mixxxbot
Copy link
Collaborator Author

Commented by: rafvdm
Date: 2015-05-06T15:38:07Z


Also note https://bugs.launchpad.net/mixxx/+bug/1452325 as I think they may be related

@mixxxbot
Copy link
Collaborator Author

Commented by: esbrandt
Date: 2015-05-09T14:43:27Z


@raf
If you deactivate the keyboard shortcuts, the CONFIGURABLE shortcuts for controlling Mixxx features are deactivated.
http://mixxx.org/manual/1.12/chapters/appendix.html#keyboard-mapping-table
http://mixxx.org/wiki/doku.php/mixxxcontrols

The standard application shortcuts in the menu on top of the application window are not deactivated.
http://mixxx.org/manual/1.12/chapters/appendix.html#application-shortcuts

@mixxxbot
Copy link
Collaborator Author

Commented by: rafvdm
Date: 2015-05-10T15:59:34Z


OK, I can live with that.  ;-)

Just one more remark, however ...
If this is a standard application shortcut, shouldn't it be marked as such in the drop down list?

@mixxxbot
Copy link
Collaborator Author

Commented by: rafvdm
Date: 2015-05-11T07:51:50Z


"Space" does not show as shortcut key for Maximize Library when running r5431 with Ubuntu 14.04 64-bit.
It does show correctly however when running r5418 with Windows 7 64-bit.

I will install most recent releases as soon as I can, but I am wondering...
It's OK in r5431, but not OK in the less recent r5418?
Are the release numbers not to be trusted across platforms, perhaps?

@mixxxbot
Copy link
Collaborator Author

Commented by: ywwg
Date: 2015-05-12T22:12:52Z


I've noticed the "Space" indicator doesn't show up when mixxx is in windowed mode, but does at fullscreen. I'm guessing this is an Ubuntu app-menu bug, not a mixxx bug.

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2015-05-20T20:34:35Z


This control is not doing anything. I tried mapping it to a button press. This is what I see in the debugging output:

Debug [Controller]: "MIDI status 0x90 (ch 1, opcode 0x9), ctrl 0x2C, val 0x7F" 
Warning [Controller]: ControlDoublePrivate::getControl returning NULL for ( "[Master]" , "maximize_library" ) 
Debug [Controller]: "MIDI status 0x90 (ch 1, opcode 0x9), ctrl 0x2C, val 0x00" 
Warning [Controller]: ControlDoublePrivate::getControl returning NULL for ( "[Master]" , "maximize_library" )

In the XML mapping file, there is:
            <control>
                <group>[Master]</group>
                <key>maximize_library</key>
                <description>MIDI Learned from 2 messages.</description>
                <status>0x90</status>
                <midino>0x2C</midino>
                <options>
                    <normal/>
                </options>
            </control>

@mixxxbot
Copy link
Collaborator Author

Commented by: Be-ing
Date: 2015-05-21T17:32:46Z


Nevermind that, the issue was resolved after "scons -c prefix=/local install && scons -c && scons prefix=/local install".

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Released.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.0.0 milestone Aug 24, 2022
@ronso0 ronso0 added skins and removed skin labels Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants