Review Request: Add a button to delete sessions in the SessionChooserDialog

Vishesh Handa handa.vish at gmail.com
Sat Mar 3 08:42:28 UTC 2012


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

(Updated March 3, 2012, 8:42 a.m.)


Review request for KDevelop.


Changes
-------

Added code to make sure the button isn't shown for the last row (Create new session)


Description
-------

I've added a very discreet button on the right of the tree view. It is shown after a delay of 500 ms.

Unfortunately, I had some minor issues with the button's placement. I've managed to fix them manually, and I've added a TODO.


Diffs (updated)
-----

  shell/sessioncontroller.cpp 42a2128 

Diff: http://git.reviewboard.kde.org/r/104143/diff/


Testing
-------

Works for me!


Screenshots
-----------


  http://git.reviewboard.kde.org/r/104143/s/445/


Thanks,

Vishesh Handa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120303/82ff048e/attachment.html>


More information about the KDevelop-devel mailing list