GwenviewProject
ActivePublic

Details

Description

Gwenview is KDE's simple yet powerful image viewer.

Quick links to Bugzilla tickets:

Recent Activity

Sat, Mar 16

ngraham added a comment to D13901: Touch support for Gwenview.

No worries, and thanks for the attempt! Once they're in, that'll really help other QWidgets-based KDE apps.

Sat, Mar 16, 8:52 PM · Gwenview
steffenh added a comment to D13901: Touch support for Gwenview.

I started to transfer the gestures to the framework, but I cannot promise anything.

Sat, Mar 16, 5:41 PM · Gwenview
ngraham added a comment to D13901: Touch support for Gwenview.

I imagine by adding what's here into the framework first, and then later raising the Gwenview frameworks dependency high enough and using them from there framework.

Sat, Mar 16, 4:18 PM · Gwenview
sander added a comment to D13901: Touch support for Gwenview.

Thanks for the patch. How is the gesture code going to get into KWidgetAddons now?

Sat, Mar 16, 2:44 PM · Gwenview
ngraham closed D13901: Touch support for Gwenview.
Sat, Mar 16, 10:52 AM · Gwenview
ngraham accepted D13901: Touch support for Gwenview.

Thanks so much for your patience on this. I think it's ready to land now!

Sat, Mar 16, 10:51 AM · Gwenview

Sun, Mar 10

steffenh updated the diff for D13901: Touch support for Gwenview.

Move the gestureEvent(QGestureEvent* event) and event(QEvent* event) functions from documentview.cpp and thumbnailview.cpp to the Touch class.
I think this is now ready to land.

Sun, Mar 10, 8:46 AM · Gwenview

Sat, Mar 9

ngraham added a comment to D13901: Touch support for Gwenview.

Hi @ngraham

+1 to adding gesture recognizers to KWidgetsAddons

I agree. @steffenh, any chance you'd be interested in that? Then, we could use these very nice gestures in Okular and Dolphin too. Since you've already done the hard work of de-duplicating the code and putting it into helper files, hopefully that shouldn't be too much effort, right?

https://cgit.kde.org/kwidgetsaddons.git

I was thinking over this in the last days, but I'm not sure my coding is up to the task. Perhaps after I have finished this patch I can try it.
But with this idea in the background, I'm beginning to complete remove the event and gestureEvent function in dokumentview.cpp and thumbnailview.cpp, and use the Signal / Slot mechanics from Qt .

Sat, Mar 9, 4:18 PM · Gwenview
steffenh added a comment to D13901: Touch support for Gwenview.

+1 to adding gesture recognizers to KWidgetsAddons

I agree. @steffenh, any chance you'd be interested in that? Then, we could use these very nice gestures in Okular and Dolphin too. Since you've already done the hard work of de-duplicating the code and putting it into helper files, hopefully that shouldn't be too much effort, right?

https://cgit.kde.org/kwidgetsaddons.git

Sat, Mar 9, 7:46 AM · Gwenview

Sun, Mar 3

ngraham added a comment to D13901: Touch support for Gwenview.

+1 to adding gesture recognizers to KWidgetsAddons

Sun, Mar 3, 9:24 PM · Gwenview
volkov added a comment to D13901: Touch support for Gwenview.

+1 for adding gesture recognizers to KWidgetsAddons

Sun, Mar 3, 8:56 PM · Gwenview
steffenh updated the diff for D13901: Touch support for Gwenview.
  • move some of the duplicate code in documentview and thumbnailview to functions in touch and touch_helper
  • fix some typos
  • fix copyright
Sun, Mar 3, 5:22 PM · Gwenview

Sat, Mar 2

sandsmark added a comment to D17732: Respect animation settings.

As for the discussion above, I don't see how plasma-integration is relevant. It does control QStyleHints, but you're not using any of that.
Everything you use here comes from the QStyle which is all from Breeze.

Sat, Mar 2, 7:40 PM · Gwenview

Wed, Feb 27

ngraham added inline comments to D13901: Touch support for Gwenview.
Wed, Feb 27, 2:18 PM · Gwenview
steffenh added inline comments to D13901: Touch support for Gwenview.
Wed, Feb 27, 1:17 PM · Gwenview

Sun, Feb 24

ngraham added a comment to D13901: Touch support for Gwenview.

Thanks, that makes sense. However, it might also make sense to add generic support for all of this into Qt itself, or at least into a KDE Framework (KWidgetsAddons maybe?). There are lots of other QWidgets-based KDE apps that could benefit from this stuff too! Dolphin and Okular immediately come to mind.

Sun, Feb 24, 4:20 PM · Gwenview
steffenh added a comment to D13901: Touch support for Gwenview.

From a high level perspective, I share @rkflx's concern with the amount of duplicate code. ThumbnailView::viewportEvent() is almost entirely duplicate code from DocumentView::event(), for example. Can we refactor this into re-usable functions in touch or touch_helper or something?

I will have a look at this, perhaps I can move same code around.

Sun, Feb 24, 7:12 AM · Gwenview

Fri, Feb 22

ngraham closed D14583: [MainWindow] Navigate through mouse forward/back buttons.
Fri, Feb 22, 4:41 PM · Gwenview
ngraham accepted D14583: [MainWindow] Navigate through mouse forward/back buttons.

Thanks, this works great now!

Fri, Feb 22, 4:39 PM · Gwenview
anthonyfieroni updated the diff for D14583: [MainWindow] Navigate through mouse forward/back buttons.

Test now, i can't reproduce behavior mentioned above.

Fri, Feb 22, 5:55 AM · Gwenview

Thu, Feb 21

ngraham added a comment to D14583: [MainWindow] Navigate through mouse forward/back buttons.

If someone steal events it's not problem of the patch, similar patch as D15612 should be applied for that view.

Thu, Feb 21, 9:58 PM · Gwenview
ngraham accepted D13901: Touch support for Gwenview.

Thanks @steffenh. I went back and re-tested everything and it all works *flawlessly* for me. Very impressive work. I really hope we can get it in for KDE Applications 19.04. I'll start on the code review:

Thu, Feb 21, 7:35 PM · Gwenview
steffenh updated the diff for D13901: Touch support for Gwenview.

rebase

Thu, Feb 21, 9:01 AM · Gwenview

Feb 20 2019

ngraham closed D19169: Compile tests files with enabled exceptions, required for exiv2 0.27.
Feb 20 2019, 4:22 PM · Gwenview
ngraham added a comment to D19169: Compile tests files with enabled exceptions, required for exiv2 0.27.

Oh no, you didn't do anything wrong at all. It's a Phabricator bug that your authorship information gets discarded when if you use git format-patch (Phab wants everyone to use arc; see https://community.kde.org/Infrastructure/Phabricator#Using_Arcanist).

Feb 20 2019, 4:20 PM · Gwenview
tgurr added a comment to D19169: Compile tests files with enabled exceptions, required for exiv2 0.27.

Oh dear, sorry maybe I did something wrong uploading the git format patch. Timo Gurr <timo.gurr@gmail.com>.

Feb 20 2019, 12:01 PM · Gwenview
ngraham accepted D19169: Compile tests files with enabled exceptions, required for exiv2 0.27.

Thanks. Can you please provide your real name and email address so I can land your patch with correct authorship information?

Feb 20 2019, 3:56 AM · Gwenview

Feb 19 2019

tgurr requested review of D19169: Compile tests files with enabled exceptions, required for exiv2 0.27.
Feb 19 2019, 8:05 PM · Gwenview
ngraham closed D19121: Allow Gwenview to be able to open .kra (Krita) files.
Feb 19 2019, 3:40 PM · Gwenview
ngraham added a comment to D19121: Allow Gwenview to be able to open .kra (Krita) files.

Sweet, will add the dep in another patch.

Feb 19 2019, 3:33 PM · Gwenview
sitter accepted D19121: Allow Gwenview to be able to open .kra (Krita) files.

FTR: kra support comes from kimageformats plugins to qimage. so do other already listed mimetypes though - it's fine.

Feb 19 2019, 2:51 PM · Gwenview

Feb 18 2019

ngraham updated the diff for D19121: Allow Gwenview to be able to open .kra (Krita) files.

Rebase on current master

Feb 18 2019, 8:50 PM · Gwenview
volkov closed D19122: Set the same image mime types in all desktop files.
Feb 18 2019, 8:22 PM · Gwenview
ngraham accepted D19122: Set the same image mime types in all desktop files.

Clever! Works great.

Feb 18 2019, 5:32 PM · Gwenview
volkov added a comment to D18852: Add mime types for raw image formats to desktop file.

D19122

Feb 18 2019, 5:23 PM · Gwenview
volkov added a reviewer for D19122: Set the same image mime types in all desktop files: Gwenview.
Feb 18 2019, 5:23 PM · Gwenview
volkov requested review of D19122: Set the same image mime types in all desktop files.
Feb 18 2019, 5:23 PM · Gwenview
ngraham added a comment to D18852: Add mime types for raw image formats to desktop file.

Just noticed this: should similar support be added into gvpart.desktop too?

Feb 18 2019, 4:28 PM · Gwenview
ngraham updated the summary of D19121: Allow Gwenview to be able to open .kra (Krita) files.
Feb 18 2019, 4:20 PM · Gwenview
ngraham updated the test plan for D19121: Allow Gwenview to be able to open .kra (Krita) files.
Feb 18 2019, 4:00 PM · Gwenview
ngraham requested review of D19121: Allow Gwenview to be able to open .kra (Krita) files.
Feb 18 2019, 3:59 PM · Gwenview
volkov closed D18852: Add mime types for raw image formats to desktop file.
Feb 18 2019, 2:58 PM · Gwenview
ngraham accepted D18852: Add mime types for raw image formats to desktop file.
Feb 18 2019, 2:43 PM · Gwenview
volkov added a comment to D18852: Add mime types for raw image formats to desktop file.

ping

Feb 18 2019, 9:39 AM · Gwenview

Feb 13 2019

volkov updated the summary of D18852: Add mime types for raw image formats to desktop file.
Feb 13 2019, 2:46 PM · Gwenview

Feb 12 2019

volkov updated the diff for D18852: Add mime types for raw image formats to desktop file.

modify existing desktop file instead of installing a new one

Feb 12 2019, 5:30 PM · Gwenview

Feb 11 2019

ngraham added a comment to D13901: Touch support for Gwenview.

@steffenh thanks for your patience here, and I'm sorry this hasn't landed yet. Could you rebase it on current master please? I'll redouble my efforts to test and rustle up some code reviewers for you.

Feb 11 2019, 8:19 PM · Gwenview
volkov added a comment to D18852: Add mime types for raw image formats to desktop file.

Wouldn't it be better to add mime types to org.kde.gwenview.desktop during the build?
https://lxr.kde.org/ doesn't respond, so I can't find an example for now.

Feb 11 2019, 3:09 PM · Gwenview
ngraham requested changes to D18852: Add mime types for raw image formats to desktop file.
Feb 11 2019, 2:52 PM · Gwenview
ngraham added a comment to D18852: Add mime types for raw image formats to desktop file.

And the added gwenviewApplication_raw.desktop contains translations because it is just a modified copy of org.kde.gwenview.desktop.

Feb 11 2019, 2:51 PM · Gwenview