CTRL+P doesn't start Preferences

Bug #1922961 reported by OsZ
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Medium
Unassigned

Bug Description

With Mixxx 2.3.0-beta (build r8110 - confirmed with git-2.3-2021-04-07) the Preferences Window doesn't open anymore using CTRL+P

Expected: CTRL+P should open the Preferences Window

compiled from source using EndevourOS (Arch based)
cmake v 3.20.0
gcc v 10.2.0

Tags: i18n packaging
Revision history for this message
ronso0 (ronso0) wrote :

maybe a system shortcut is catching the signal and it's not registered in Mixxx?

what happens in other apps or on the desktop when you press Ctrl+P ?

Revision history for this message
OsZ (toszlanyi) wrote :

CTRL+P is the system wide print command

In previous versions of Mixxx 2.3 beta as well as on current main branch (2.4 alpha) it is available and opens Preferences.

While comparing the versions I started them with option --locale en and then Ctrl+P is working. I further tried:

mixxx --locale fr | Ctrl+P is working
mixxx --locale nl | Ctrl+P is working
mixxx --locale de | Ctrl+P is NOT working and the shortcut is also not visible inside the pull down menu.

Revision history for this message
jus (jus) wrote :

I noticed your screenshot does not show the shortcuts from the Mixxx translation files for the German locale (de)
Are they overridden by your system?
According to your systems setting, are you using EN keyboard layout as input source by chance?

In the German Mixxx locale ``Ctrl`` is translated with ``Strg``, ``Shift`` with ``Umschalt`` etc.
(see my screenshot from a recent Windows installation of Mixxx).

Revision history for this message
OsZ (toszlanyi) wrote :

@jus - i see where you coming from - but the words translation in my Mixxx installation is German, except the Ctrl+...

Is there a special flag to compile with or dependencies I need? My system locale is set to German

$ locale
LANG=de_DE.UTF-8
LC_CTYPE="de_DE.UTF-8"
LC_NUMERIC=de_DE.UTF-8
LC_TIME=de_DE.UTF-8
LC_COLLATE="de_DE.UTF-8"
LC_MONETARY=de_DE.UTF-8
LC_MESSAGES="de_DE.UTF-8"
LC_PAPER=de_DE.UTF-8
LC_NAME=de_DE.UTF-8
LC_ADDRESS=de_DE.UTF-8
LC_TELEPHONE=de_DE.UTF-8
LC_MEASUREMENT=de_DE.UTF-8
LC_IDENTIFICATION=de_DE.UTF-8
LC_ALL=

Revision history for this message
jus (jus) wrote :

This is weird, never noticed that strange behavior. But I can confirm similar issues with macOS.

CTRL+P is no issue in macOS, because it is overridden by the system wide preferences shortcut ⌘, (Unicode: U+2318).

But CTRL+F (search focus) does not work either, if the language is set to German.

$ locale
LANG="de_DE.UTF-8"
LC_COLLATE="de_DE.UTF-8"
LC_CTYPE="de_DE.UTF-8"
LC_MESSAGES="de_DE.UTF-8"
LC_MONETARY="de_DE.UTF-8"
LC_NUMERIC="de_DE.UTF-8"
LC_TIME="de_D

I can reproduce that back to Mixxx 2.2.1, Mixxx 2.1.8 being the last working version.

Moreover, if running

./mixxx --developer

You can see one press of CTRL+F is fired multiple time
debug [Main] keyboard press: "Ctrl+F"
debug [Main] keyboard press: "Ctrl+F"
debug [Main] keyboard press: "Ctrl+F"
debug [Main] keyboard press: "Ctrl+F"

This needs further investigation.

tested with 2.4.0-alpha-pre (build main r7832), and 2.3.0-beta (build 2.3 r8120)

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

The issue happened before elsewhere, after translating shortcut strings.
We might miss a QT dependency for translating qt base itself.
See [1], [2] and [5].

As last resort, we might just disable translations for keyboard shortcut strings in the translation template.

[1] https://github.com/void-linux/void-packages/issues/14415
[2] https://github.com/helloSystem/ISO/issues/9
[3] https://archlinux.org/packages/extra/any/qt5-translations/

Revision history for this message
OsZ (toszlanyi) wrote :

Wicked!

... installed `qt5-translations` and the German translated Mixxx GUI now shows "Strg+P" and shortcut works as expected.

So the dependency to `qt5-translations` will solve it. Is that a Mixxx or Arch package issue?

Thanks a lot for your help!

Revision history for this message
ronso0 (ronso0) wrote :

a web search reveals that qt5-translations is an Arch package only.
so it's not an actual build dependency..
maybe just tell the package maintainer about it.

Revision history for this message
OsZ (toszlanyi) wrote :

I left a comment there - Jan Holzhaus is the maintainer ;)

Revision history for this message
jus (jus) wrote :

This bug does NOT apply to Windows, there we pack the needed ``qtbase_`` catalog [1] correctly [5].
The catalog is in the qttranslations-everywhere-src-5.xx.x.zip submodule, part of a default qt installation.

We need to add this to the build requirements for the other supported platforms too.

[1] https://doc.qt.io/qt-5/internationalization.html#locating-qt-translations
[2] https://github.com/mixxxdj/mixxx/blob/2.3/CMakeLists.txt#L1416

Changed in mixxx:
milestone: none → 2.3.0
tags: added: i18n packaging
Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

Just noticed that Fedora is also affected. Will be fixed with the next builds.

https://github.com/rpmfusion/mixxx/commit/29c7a3d98b4f55824f20617adc9cff043192e3c5

Revision history for this message
ronso0 (ronso0) wrote :

so, this is now fixed on Arch an Fedora?

Revision history for this message
OsZ (toszlanyi) wrote :

For Arch nothing changed yet. Anyhow, I think people could find my comment and install the package manually in case they have issues.

https://aur.archlinux.org/packages/mixxx_beta-git/#comment-800966

Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

Fix for Fedora.

Revision history for this message
Jan Holthuis (holthuis-jan) wrote :

Oh, didn't notice that comment. I'll add the missing dependency.

Revision history for this message
Jan Holthuis (holthuis-jan) wrote :
Revision history for this message
Jan Holthuis (holthuis-jan) wrote :
Changed in mixxx:
status: Confirmed → Fix Committed
Revision history for this message
OsZ (toszlanyi) wrote :

Does it still need a fix for macOS? See comment #5 by @jus:
https://bugs.launchpad.net/mixxx/+bug/1922961/comments/5

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/10389

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.