D13444: Update to display the name of the session in the "Delete Session" confirmation dialogue box
Christoph Roick
noreply at phabricator.kde.org
Wed Jun 20 21:25:23 UTC 2018
croick added a comment.
> Discard unrelated change
> Anything else needs to be done?
Could you reinsert the (additional) trailing line to restrict the commit to what it's supposed to solve?
Also it would be nice to have a short comment in the code, explaining why the last 3 characters have to be removed. Since that is not obvious from the code.
REPOSITORY
R32 KDevelop
REVISION DETAIL
https://phabricator.kde.org/D13444
To: sagnikchaudhuri, #kdevelop
Cc: croick, apol, kdevelop-devel, antismap, iodelay, vbspam, njensen, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20180620/9bbfee6b/attachment.html>
More information about the KDevelop-devel
mailing list