D17978: QuickOpenSession: Don't offer the already active session

Dominik Haumann noreply at phabricator.kde.org
Sat Jan 5 12:53:55 GMT 2019


dhaumann added a comment.


  I would prefer to not hide it since it's confusing. Either gray out the session by calling action ->setEnabled(false), or mark it as active by e.g. adding a '*' or similar to the name.
  
  In any case, using the range based for loop is certainly bette.

REPOSITORY
  R40 Kate

REVISION DETAIL
  https://phabricator.kde.org/D17978

To: loh.tar, #kate
Cc: dhaumann, kwrite-devel, #kate, hase, michaelh, ngraham, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20190105/ea22c261/attachment.html>


More information about the KWrite-Devel mailing list