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

Reply via email to