D20198: Don't lose list position after installing KNS cursor themes

Nathaniel Graham noreply at phabricator.kde.org
Tue Apr 2 17:12:35 BST 2019


ngraham added a comment.


  Why do we need this do-it-once helper function?

INLINE COMMENTS

> kcmcursortheme.cpp:466
> +            const CursorTheme *theme = selectedIndex().isValid() ? m_proxyModel->theme(selectedIndex()) : nullptr;
> +            if(theme) {
> +                QString currentTheme = theme->name();

Coding style: space between `if` and `(`

REPOSITORY
  R119 Plasma Desktop

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

To: leinir, #plasma, ngraham, broulik
Cc: plasma-devel, #plasma, jraleigh, 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/20190402/1c4250cc/attachment.html>


More information about the Plasma-devel mailing list