-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125548/
-----------------------------------------------------------

Review request for Kdenlive.


Repository: kdenlive


Description
-------

This proposed patch improves the configuration dialog UI for the Jog/Shuttle 
tab: the texts for actions that can be assigned to buttons of a jog/shuttle 
device previously contained "&". These ampersands cause action texts to be 
sorted in an unintuitive way that makes finding the correction action hard. 
This patch removes all ampersands from the action descriptive texts, so that 
the resulting action list is sorted according to user expectations. In 
particular, leading ampersands don't cause action names to get sorted into two 
sets: one going from &A-&Z and another one going from A-Z. With this patch, 
there is only a single sorted list A-Z without any distracting ampersands.

On a sidenote: Kdenlive up to 0.9.10 already removed the ampersands, only later 
they got in again during transition to KF5.


Diffs
-----

  src/mainwindow.cpp e147e93 

Diff: https://git.reviewboard.kde.org/r/125548/diff/


Testing
-------

Yes; on Kdenlive Applications/15.08 with a ShuttlePRO v2 jog/shuttle device. 
German locale set. Action descriptive texts now appear without interspersed 
ampersands (&). Actions still correctly trigger and button-to-action 
assignments can successfully be changed and work correctly.


Thanks,

The DiveO

_______________________________________________
kdenlive mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kdenlive

Reply via email to