Reintroduce QML on 2.7 alpha and rename the cryptic QML argument #15513
Conversation
This reverts commit c37ecda.
acolombier
force-pushed
the
chore/rename-qml-arg
branch
from
d75430f to
55e90ca
Compare
| forUserFeedback ? QCoreApplication::translate("CmdlineArgs", | ||
| "Loads experimental QML GUI instead of legacy QWidget skin") | ||
| : QString()); | ||
| const QCommandLineOption qml(QStringLiteral("next"), |
There was a problem hiding this comment.
I was not involved in the discussion, but isn't --next even more cryptic? Could basically mean everything (e.g., "play next track").
If you think it's too technical, maybe consider --new-user-interface or something like that?
Also, keep in mind that contributors will likely still use the term "QML" to refer to the new UI, so I am wondering if it's really that cryptic after all.
I'm happy with whatever name you think fits, I just wanted to point this out.
There was a problem hiding this comment.
Yeah, that's a good shout, I didn't think of the "Next" in the context of a player which is very relevant to Mixxx. I think --new-user-interface is a good shout!
Also, keep in mind that contributors will likely still use the term "QML" to refer to the new UI, so I am wondering if it's really that cryptic after all.
Note that --qml will continue working, at least for some time! But technically, this remains implementation details; the fact that contributors or technical-aware people continue referencing it is this way is fine tho - I surely will continue calling it QML :)
However, for less technical user, this feels unnecessary confusing, especially now that QML is also used in the legacy UI (for controller offscreen rendering)
2 participants
This addresses #15437 and honour the agreed plan regarding QML feature on
main