D7331: Show UI for per screen scaling options on supported platforms

Kai Uwe Broulik noreply at phabricator.kde.org
Tue Aug 15 15:30:29 UTC 2017


broulik added inline comments.

INLINE COMMENTS

> outputconfig.cpp:139
> +        mScale->addItem(i18nc("Scale multiplier, show everything at 2 times normal scale", "2x"), 2);
> +        connect(mScale, static_cast<void(QComboBox::*)(int)>(&QComboBox::currentIndexChanged),
> +                this, &OutputConfig::slotScaleChanged);

Use `QComboBox::activated` which only fires when the user explicitly selects an option, should also remove the need for blocking signals above.

REPOSITORY
  R104 KScreen

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

To: davidedmundson
Cc: broulik, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170815/05c036e3/attachment.html>


More information about the Plasma-devel mailing list