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