D7996: Properly create KPixmapSequence
Albert Astals Cid
noreply at phabricator.kde.org
Tue Sep 26 20:21:11 UTC 2017
aacid accepted this revision.
aacid added a comment.
This revision is now accepted and ready to land.
Whoever decided to change from iconName to fullpath deserves some punishment for having a class constructor that has exactly signature but behaves totally different.
every single instance is broken
https://lxr.kde.org/source/frameworks/kpeople/src/widgets/mergedialog.cpp#0082
https://lxr.kde.org/source/frameworks/knewstuff/src/uploaddialog.cpp#0147
https://lxr.kde.org/source/calligra/krita/libs/ui/widgets/kis_cie_tongue_widget.cpp#0162
https://lxr.kde.org/source/extragear/base/nepomuk-webminer/src/lib/ui/fetcherdialog.cpp#0140
meh, i'll forget this to make my life better.
REPOSITORY
R223 Okular
BRANCH
Applications/17.08
REVISION DETAIL
https://phabricator.kde.org/D7996
To: elvisangelaccio, #okular, aacid
Cc: aacid, #okular
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20170926/734ab4e7/attachment.html>
More information about the Okular-devel
mailing list