D8115: Port KAccess away from kdelibs4support
Aleix Pol Gonzalez
noreply at phabricator.kde.org
Mon Oct 2 23:45:59 UTC 2017
apol added inline comments.
INLINE COMMENTS
> davidedmundson wrote in kaccess.cpp:722
> we've effectively removed this connect to dialogClosed. Now closing goes to noClicked.
>
> Is this a problem?
I noticed. I'm not sure to be honest.
My thinking is that the user shouldn't notice the difference between saying "No" and closing the window.
I can short-circuit the QDialogButtonBox::rejected if that makes you feel more comfortable with the patch. :)
REPOSITORY
R119 Plasma Desktop
REVISION DETAIL
https://phabricator.kde.org/D8115
To: apol, #plasma
Cc: davidedmundson, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171002/b7d98e9d/attachment.html>
More information about the Plasma-devel
mailing list