Mon, Jun 10
Sun, Jun 9
Honestly i'd prefer if we wait for 5.10 and then use QMetaObject::invokeMethod instead of QTimer::singleShot.
Mon, Jun 3
Sun, Jun 2
Sat, Jun 1
Need to postpone this until end of June.
Tue, May 28
Spectacle also supports both. We could probably do the same here in the meantime. I'm fine with replacing the current toolbar button with this though; it's much more useful, and Kipi still gets its own menu. Please also add the new action's to the file menu, context menu, and Operations panel. That way it's still accessible for people who don't want it in the toolbar.
Sun, May 26
By renaming the config name in gwenviewconfig.kcfg, you'll reset existing users' preferences. Whenever you want to rename a key in a config file, you need to ship a kconfig update script to update existing users' config files. Look at the contents of the kconf_update folder in the source tree for an example of an existing one. And here's the documentation: https://techbase.kde.org/Development/Tools/Using_kconf_update
All of these new variables can be const
Sat, May 25
...also a blurred thumbnail suggests that the image is defocused.
I haven't checked which folder Gwenview actually tries to delete, but the thumbnails are stored in ~/.cache/thumbnails instead of ~/.thumbnails.
Fri, May 24
You guys really have to setup clang-format or astyle to automate those whitespace formatting issues, IMO.
Thanks, just one last little thing...
Fix unit test.
Sry, I missed that.
It's on my to-do list, I'll take a look tomorrow!
Thu, May 23
Thanks. To account for this behavioral change, I think we need to adjust the UI strings in the settings window as well. Did my suggestion above make sense?
Ahh whatever, this is fine as-is then.
Sry, I should really enable Kate's spell checking.
Thanks in advance for fixing this minor typo.
Docs updated. Need to apply https://phabricator.kde.org/D21329 first, to avoid conflict.