D23412: Fix ascending/descending choosers getting unchecked when re-selecting the same sort order

[Méven Car]

meven added a comment.


  A simpler alternative could be to use m_sortByActions, I have tested it.
  
    foreach (QAction* groupAction, m_sortByActions.values()) {
        KActionMenu* actionMenu = qobject_cast<KActionMenu*>(groupAction);
        if (actionMenu) {
            foreach (QAction* subAction, actionMenu->menu()->actions()) {
                subAction->setChecked(false);
            }
        } else if (groupAction->actionGroup()) {
            groupAction->setChecked(false);
        }
    }

REPOSITORY
  R318 Dolphin

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

To: ngraham, elvisangelaccio, #dolphin
Cc: meven, kfm-devel, aprcela, vmarinescu, fprice, MrPepe, fbampaloukas, alexde, Codezela, feverfew, spoorun, navarromorales, firef, andrebarros, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20190825/e2bd2cc0/attachment.htm>