https://bugs.kde.org/show_bug.cgi?id=411223
Nate Graham changed:
What|Removed |Added
Latest Commit||https://commits.kde.org/dol
|
https://bugs.kde.org/show_bug.cgi?id=411223
--- Comment #8 from Gabriel Fernandes ---
But it seems to be missing a parenthesis at the end of the for, in the diff. I
had added one before the curly at the end in my local build.
for (QAction *groupAction : qAsConst(m_sortByActions) ->)<- {
--
You
https://bugs.kde.org/show_bug.cgi?id=411223
--- Comment #7 from Nate Graham ---
Still in patch review, I'm afraid.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=411223
--- Comment #6 from Gabriel Fernandes ---
I thought it would land in Dolphin Version 19.08.1
I tried the patch myself, it seems to work
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=411223
--- Comment #5 from Gabriel Fernandes ---
great to hear
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=411223
--- Comment #4 from Nate Graham ---
Filed a patch: https://phabricator.kde.org/D23412
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=411223
Nate Graham changed:
What|Removed |Added
Resolution|WAITINGFORINFO |---
Status|NEEDSINFO
https://bugs.kde.org/show_bug.cgi?id=411223
--- Comment #2 from Gabriel Fernandes ---
sure, https://youtu.be/TKQrFNcf9Qw
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=411223
Nate Graham changed:
What|Removed |Added
CC||n...@kde.org
Resolution|---