GwenviewProject
ActivePublic

Details

Description

Gwenview is KDE's simple yet powerful image viewer.

Quick links to Bugzilla tickets:

Recent Activity

Yesterday

abetts added a comment to D18402: Use semantically correct restoration icon.

+1

Sun, Jan 20, 11:12 PM · Gwenview
shubham closed D18402: Use semantically correct restoration icon.
Sun, Jan 20, 5:48 PM · Gwenview
shubham closed D18391: Use user-trash icon for Trash action.
Sun, Jan 20, 5:47 PM · Gwenview
ngraham added reviewers for D18403: Shortcut to focus navigator bar: Gwenview, KDE Applications, cfeck.
Sun, Jan 20, 5:44 PM · Gwenview
ngraham accepted D18391: Use user-trash icon for Trash action.
Sun, Jan 20, 5:43 PM · Gwenview
shubham retitled D18391: Use user-trash icon for Trash action from user-trash icon only to Use user-trash icon for Trash action.
Sun, Jan 20, 5:36 PM · Gwenview
shubham updated the summary of D18391: Use user-trash icon for Trash action.
Sun, Jan 20, 5:34 PM · Gwenview
ngraham accepted D18402: Use semantically correct restoration icon.

Sounds like we need to fix that icon. We'll do that in the breeze-icons repo.

Sun, Jan 20, 5:32 PM · Gwenview
shubham added a comment to D18402: Use semantically correct restoration icon.

@ngraham R318:ca25af6d38c8d7c4ac89f83d3b359500fb65b2fa

Sun, Jan 20, 5:30 PM · Gwenview
ngraham added a reviewer for D18402: Use semantically correct restoration icon: VDG.

Hmm, I'm not sure about this. edit-undo isn't really that semantically incorrect since restoring a trashed item can be considered a form of undo. Also from a practical perspective, the edit-undo icon is far more meaningful than this restoration icon, whose meaning is not at all clear.

Sun, Jan 20, 5:24 PM · Gwenview
ngraham added a comment to D18391: Use user-trash icon for Trash action.

Great, thanks. Now please fix the commit message to be a statement in the imperative mood (i.e. "do something to that other thing"). See https://chris.beams.io/posts/git-commit/#seven-rules

Sun, Jan 20, 5:22 PM · Gwenview
shubham updated subscribers of D18403: Shortcut to focus navigator bar.
Sun, Jan 20, 7:38 AM · Gwenview
shubham planned changes to D18403: Shortcut to focus navigator bar.
Sun, Jan 20, 7:38 AM · Gwenview
shubham requested review of D18403: Shortcut to focus navigator bar.
Sun, Jan 20, 7:36 AM · Gwenview
shubham updated the test plan for D18402: Use semantically correct restoration icon.
Sun, Jan 20, 4:16 AM · Gwenview
shubham requested review of D18402: Use semantically correct restoration icon.
Sun, Jan 20, 4:16 AM · Gwenview
shubham updated the test plan for D18391: Use user-trash icon for Trash action.
Sun, Jan 20, 4:09 AM · Gwenview
shubham updated the test plan for D18391: Use user-trash icon for Trash action.
Sun, Jan 20, 4:02 AM · Gwenview

Sat, Jan 19

ngraham added a comment to D18391: Use user-trash icon for Trash action.

Does this screenshot actually show the place where the icon is used? ;-)

Sat, Jan 19, 7:29 PM · Gwenview
shubham updated the test plan for D18391: Use user-trash icon for Trash action.
Sat, Jan 19, 7:25 PM · Gwenview
ngraham added a comment to D18391: Use user-trash icon for Trash action.

Did you test by running QT_SCALE_FACTOR=2 gwenview with this patch? :)

Sat, Jan 19, 5:39 PM · Gwenview
shubham requested review of D18391: Use user-trash icon for Trash action.
Sat, Jan 19, 5:12 PM · Gwenview

Wed, Jan 9

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

Some of these gesture recognizers would be very helpful in other KDE programs as well (Okular is what I have in mind). Can they be moved to some central place where both Gwenview and Okular can use them?

Wed, Jan 9, 10:34 AM · Gwenview

Mon, Jan 7

ngraham closed D6083: Bump max thumbnail size to 512px.
Mon, Jan 7, 12:17 AM · Gwenview

Sun, Jan 6

gateau accepted D6083: Bump max thumbnail size to 512px.

I agree, unless the XDG spec evolves, it is becoming more and more useless anyway.

Sun, Jan 6, 1:23 PM · Gwenview
bruns closed D17873: Compile more files with enabled exceptions, required for exiv2 0.27.
Sun, Jan 6, 3:35 AM · Gwenview
ngraham accepted D17873: Compile more files with enabled exceptions, required for exiv2 0.27.
Sun, Jan 6, 3:25 AM · Gwenview
bruns added a comment to D17873: Compile more files with enabled exceptions, required for exiv2 0.27.

arc does something very strange when I try to apply this patch:

Does it need a rebase on master or something?

Sun, Jan 6, 3:05 AM · Gwenview

Sat, Jan 5

ngraham added a comment to D17873: Compile more files with enabled exceptions, required for exiv2 0.27.

arc does something very strange when I try to apply this patch:

Sat, Jan 5, 4:49 AM · Gwenview

Fri, Jan 4

bruns added a comment to D17873: Compile more files with enabled exceptions, required for exiv2 0.27.

So, anyone brave enough to accept?

Fri, Jan 4, 9:56 PM · Gwenview
asturmlechner added a comment to D17873: Compile more files with enabled exceptions, required for exiv2 0.27.

Yes, this is necessary to successfully build on Gentoo as well.

Fri, Jan 4, 6:42 PM · Gwenview

Thu, Jan 3

wbauer added a comment to D17873: Compile more files with enabled exceptions, required for exiv2 0.27.

I can't comment whether this is the desired way to do it

Thu, Jan 3, 7:26 PM · Gwenview
wbauer added a comment to D17873: Compile more files with enabled exceptions, required for exiv2 0.27.

I can't comment whether this is the desired way to do it, but it does fix the build with exiv2 0.27, on openSUSE at least. (with D17872 in)
Without it, the compilation fails due to errors:
error: exception handling disabled, use -fexceptions to enable

Thu, Jan 3, 7:20 PM · Gwenview

Wed, Jan 2

bruns closed D17872: Use forward declaration for Exiv2::Image, port to std::unique_ptr.
Wed, Jan 2, 12:49 AM · Gwenview

Tue, Jan 1

bruns added reviewers for D17873: Compile more files with enabled exceptions, required for exiv2 0.27: ngraham, cfeck, asturmlechner.
Tue, Jan 1, 11:48 PM · Gwenview
ngraham accepted D17872: Use forward declaration for Exiv2::Image, port to std::unique_ptr.
Tue, Jan 1, 11:16 PM · Gwenview
bruns updated the diff for D17872: Use forward declaration for Exiv2::Image, port to std::unique_ptr.

fix indentation

Tue, Jan 1, 2:35 PM · Gwenview
bruns updated subscribers of D17872: Use forward declaration for Exiv2::Image, port to std::unique_ptr.
Tue, Jan 1, 1:50 PM · Gwenview
bruns updated subscribers of D17873: Compile more files with enabled exceptions, required for exiv2 0.27.
Tue, Jan 1, 1:49 PM · Gwenview
pino removed a reviewer for D17872: Use forward declaration for Exiv2::Image, port to std::unique_ptr: pino.
Tue, Jan 1, 2:59 AM · Gwenview
ngraham added reviewers for D17872: Use forward declaration for Exiv2::Image, port to std::unique_ptr: pino, cfeck.
Tue, Jan 1, 1:05 AM · Gwenview

Mon, Dec 31

ngraham accepted D6083: Bump max thumbnail size to 512px.

I am inclined to accept this. It's a nice enhancement that seems to work fine in my testing, and while it doesn't strictly speaking follow the letter of the XDG spec, I don't think there's any sense letting the perfect being the enemy of the good here. We all know how that game would end: tons of wrangling in the FDO world and no official support for 512px thumbnails for years while we wait. I say let's do this now.

Mon, Dec 31, 10:36 PM · Gwenview
davidedmundson added a comment to D17732: Respect animation settings.

Intuitively this patch makes sense and is certainly "technically correct", but feedback we had in plasma with the wallpaper was that a fullscreen transition is quite a bit separate to the setting you'd use for small widget effects. 100ms for altering a few pixels of a shadow is a much slower in pixels per second than 100ms to slide something across the whole screen.

Mon, Dec 31, 5:37 PM · Gwenview
ngraham updated subscribers of D17732: Respect animation settings.
Mon, Dec 31, 4:17 PM · Gwenview
ngraham added a comment to D17732: Respect animation settings.

For now this should probably look at the setting in the Widget Style, but it would be nice to unify these at tome point to a single "Animation Speed" control that affects both apps and Plasma.

Mon, Dec 31, 4:16 PM · Gwenview
ngraham added a comment to D17732: Respect animation settings.

I know that KWin has its own animation setting, exposed in the Compositor KCM. Is that the one that sets the animation speed in the Plasma Integration plugin?

Mon, Dec 31, 4:14 PM · Gwenview
sandsmark added a comment to D17732: Respect animation settings.

I'm afraid not, sorry. But @cfeck or another Frameworks or KDE Applications person may know.

Mon, Dec 31, 4:09 PM · Gwenview
sandsmark updated subscribers of D17732: Respect animation settings.
Mon, Dec 31, 3:53 PM · Gwenview

Sun, Dec 30

bruns updated the diff for D17872: Use forward declaration for Exiv2::Image, port to std::unique_ptr.

fix test build

Sun, Dec 30, 4:02 PM · Gwenview
asturmlechner abandoned D17869: Fix build with exiv2-0.27.

Abandoned in favor of better D17872

Sun, Dec 30, 3:19 PM · Gwenview