The terms "add" and "remove" are so much easier to understand than "add" and "unset". Even after using it over 50 times, I still struggle to remove a preview zone because of the "unset" term. I think Unset is a little ambiguous and not a term that editors use often. So I suggest replacing the text "Unset Preview Zone" in the Preview Render dropdown box with "Remove Preview Zone". :) Reproducible: Always Suggested while using Kdenlive git master build via ppa:kdenlive/kdenlive-master. Ubuntu 16.04 x64 w/ Unity 7.4.0 desktop environment. KDE Frameworks 5.18.0.
Patch submitted, https://git.reviewboard.kde.org/r/129056/
Dude, you are on a ROLL! :)
Nah, quick'n'easy fix, just changing two text strings. And I prefer consistent UI experience myself. :)
Git commit 58270892e65c6044eadd2c20fe0cde9a190002f7 by Jean-Baptiste Mardelle. Committed on 27/09/2016 at 21:45. Pushed by mardelle into branch 'master'. Rename "Unset/Clear preview zone to Remove..." REVIEW: 129056 M +2 -0 data/pics/CMakeLists.txt A +4 -0 data/pics/breeze-dark/dk_preview-remove-all.svg A +9 -0 data/pics/breeze-light/lt_preview-remove-all.svg M +1 -0 src/icons.qrc M +2 -2 src/mainwindow.cpp http://commits.kde.org/kdenlive/58270892e65c6044eadd2c20fe0cde9a190002f7