D24263: edit mode action
David Edmundson
noreply at phabricator.kde.org
Fri Oct 18 11:17:15 BST 2019
davidedmundson added a comment.
GB_2's comment needs fixing, two strings are out of sync.
Other than that, ship it.
INLINE COMMENTS
> GB_2 wrote in corona.cpp:502
> Missing ellipsis (...)
Given how easy it is to get these out of sync, can I suggest removing all the code to update editAction's text from Corona::setEditMode and put it in a connect here. It'll keep the editAction logic altogether.
REPOSITORY
R242 Plasma Framework (Library)
BRANCH
phab/editModeActionwq!
REVISION DETAIL
https://phabricator.kde.org/D24263
To: mart, #plasma, #vdg, ngraham
Cc: davidedmundson, broulik, GB_2, ngraham, kde-frameworks-devel, LeGast00n, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191018/76e2d012/attachment.html>
More information about the Kde-frameworks-devel
mailing list