D17533: Make the button's purpose more clear

Nathaniel Graham noreply at phabricator.kde.org
Sun Dec 16 20:37:27 GMT 2018


ngraham added inline comments.

INLINE COMMENTS

> kaccess.cpp:870
>                                     + QStringLiteral(" ") + i18n("These AccessX settings are needed for some users with motion impairments and can be configured in the KDE System Settings. You can also turn them on and off with standardized keyboard gestures.\n\nIf you do not need them, you can select \"Deactivate all AccessX features and gestures\"."));
> -
> +            questionLabel->setText("<b>" + question + "</b>");
>              KWindowSystem::setState(dialog->winId(), NET::KeepAbove);

Let's use `xi18nc()` for this:

`xi18nc("@info", "<emphasis strong='true'>%1"</emphasis>, question)`

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D17533

To: amantia, #plasma_accessibility, davidedmundson, aacid, ngraham
Cc: ngraham, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181216/29c028ab/attachment.html>


More information about the Plasma-devel mailing list