<table><tr><td style="">ngraham added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D17533">View Revision</a></tr></table><br /><div><div><p>All dialogs ask a question that can be answered with a yes or a no. The problem is that most users don't read dialog box text--especially very long text, like the kind presented here. They do generally read the dialog box button labels though, which is why it's so important that the labels indicate what the buttons do. I can understand if you don't want to do that in this patch though.</p>

<p>IMHO we should work towards allowing the button labels to be customized alongside the dialog box text. As is, this patch doesn't really improve much because the text is already so long that nobody will read it, no matter which order the strings are presented in. If there's no way to simplify the text, perhaps we could should use bold text for the important part? This is what we do for Dolphin's "permanently delete this file" dialog:<br />
<a href="https://phabricator.kde.org/F6478899" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6478899: Screenshot_20181215_140503.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R119 Plasma Desktop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17533">https://phabricator.kde.org/D17533</a></div></div><br /><div><strong>To: </strong>amantia, Plasma Accessibility, davidedmundson, aacid, ngraham<br /><strong>Cc: </strong>ngraham, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>