rkflx (Henrik Fehlauer)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Friday

  • Clear sailing ahead.

User Details

User Since
May 6 2017, 7:59 PM (66 w, 3 d)
Availability
Available

Recent Activity

Yesterday

rkflx added a comment to T9273: Announcement text for the Applications 18.08 release.

Thanks! I added the context menu back, cleaned up the blur and most importantly redacted the window title :)

Tue, Aug 14, 10:44 PM · KDE Promo, KDE Applications
rkflx added a comment to T9273: Announcement text for the Applications 18.08 release.

(You might want to use Rectangular Region, because none of the other modes can capture both a shadow and a context menu at the same time. It's a bug in Spectacle, I actually had to GIMP the screenshot for Gwenview…)

Tue, Aug 14, 10:24 PM · KDE Promo, KDE Applications
rkflx added a comment to T9273: Announcement text for the Applications 18.08 release.

I can try, the problem is that I need to blur most of it since that's full of my folders/emails, I don't have a clean demo setup handy.

Tue, Aug 14, 10:06 PM · KDE Promo, KDE Applications
rkflx committed R260:4545e1f99093: Allow to double-click to accelerate crop and red-eye modifications (authored by rkflx).
Allow to double-click to accelerate crop and red-eye modifications
Tue, Aug 14, 8:50 PM
rkflx closed D14755: Allow to double-click to accelerate crop and red-eye modifications.
Tue, Aug 14, 8:50 PM
rkflx updated the diff for D14755: Allow to double-click to accelerate crop and red-eye modifications.

confirm -> correct

Tue, Aug 14, 8:45 PM
rkflx updated subscribers of T9273: Announcement text for the Applications 18.08 release.

KMail, KDE's powerful email client, features some improvements in the travel data extraction engine. It now supports UIC 918.3 and SNCF train ticket barcodes and Wikidata-powered train station location lookup. Support for multi-traveler itineraries was added, and KMail now has integration with the KDE Itinerary app.

Tue, Aug 14, 7:20 PM · KDE Promo, KDE Applications
rkflx added a comment to T9273: Announcement text for the Applications 18.08 release.

@cfeck Sorry for not providing screenshots earlier. Was just about to capture some when I noticed your commit.

Tue, Aug 14, 7:19 PM · KDE Promo, KDE Applications
rkflx requested review of D14846: Polish Reduce Red Eye UI.
Tue, Aug 14, 4:06 PM
rkflx updated the diff for D14755: Allow to double-click to accelerate crop and red-eye modifications.
  • Update wording with my suggestion from yesterday.
  • Remove empty line.
Tue, Aug 14, 4:01 PM

Mon, Aug 13

rkflx committed R260:cdd75b6c1e16: Bump QT_MIN_VERSION and KF5_MIN_VERSION (authored by rkflx).
Bump QT_MIN_VERSION and KF5_MIN_VERSION
Mon, Aug 13, 8:36 PM
rkflx closed D14700: Bump QT_MIN_VERSION and KF5_MIN_VERSION.
Mon, Aug 13, 8:36 PM
rkflx added a comment to D14700: Bump QT_MIN_VERSION and KF5_MIN_VERSION.

Distros that are still shipping very old versions of Qt aren't going to be shipping bleeding-edge versions of Gwenview--or KDE Applications more generally.

Mon, Aug 13, 8:35 PM
rkflx added inline comments to D14755: Allow to double-click to accelerate crop and red-eye modifications.
Mon, Aug 13, 8:35 PM
rkflx added a comment to D14755: Allow to double-click to accelerate crop and red-eye modifications.

Looks good to me. Works as stated and I can't find any issues, also zooming by Ctrl+DoubleClick still works.

Mon, Aug 13, 11:19 AM
rkflx added a comment to T9358: "External" tagging for gwenview.

Hi, thanks for reaching out.

Mon, Aug 13, 10:46 AM · KPhotoAlbum, Gwenview

Sun, Aug 12

rkflx added a comment to T9273: Announcement text for the Applications 18.08 release.
In T9273#153875, @cfeck wrote:

If more time is needed for the text, I can commit it Friday night.

Sun, Aug 12, 7:20 AM · KDE Promo, KDE Applications
rkflx added a comment to D13344: Add possibility to sort by rating.

@faridb Are you still around?

Sun, Aug 12, 7:16 AM
rkflx committed R166:c20367043a2b: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
Sun, Aug 12, 6:33 AM
rkflx committed R166:6eebb57e6e1f: Fix missing notification title found by -Wswitch-enum (authored by rkflx).
Fix missing notification title found by -Wswitch-enum
Sun, Aug 12, 6:32 AM
rkflx closed D14716: Fix missing notification title found by -Wswitch-enum.
Sun, Aug 12, 6:32 AM
rkflx added a comment to D14755: Allow to double-click to accelerate crop and red-eye modifications.

(I'm aware that the size slider for Reduce Red Eye now is quite long. There will be a follow-up fixing that along with a bunch of other issues.)

Sun, Aug 12, 6:28 AM
rkflx requested review of D14755: Allow to double-click to accelerate crop and red-eye modifications.
Sun, Aug 12, 6:27 AM

Sat, Aug 11

rkflx committed R166:45d34b0ae5d4: Fix -Wswitch-default warnings (authored by rkflx).
Fix -Wswitch-default warnings
Sat, Aug 11, 10:58 PM

Fri, Aug 10

rkflx committed R166:982555d911eb: Deduplicate default filename template code (authored by rkflx).
Deduplicate default filename template code
Fri, Aug 10, 7:51 PM
rkflx added a commit to T9252: Improve default filename when saving: R166:982555d911eb: Deduplicate default filename template code.
Fri, Aug 10, 7:51 PM · Spectacle
rkflx closed D14588: Deduplicate default filename template code.
Fri, Aug 10, 7:51 PM
rkflx removed projects from T9273: Announcement text for the Applications 18.08 release: Spectacle, Gwenview.

Thank you everyone who wrote and improved the text! It's in a pretty good state now as far as I could tell when checking thoroughly (at least for the apps that are present in the etherpad, we might want to expand next time…).

Fri, Aug 10, 7:40 PM · KDE Promo, KDE Applications
rkflx added a comment to T7927: Fonts “easy mode”.

@progwolff Any comments on the proposal below?

Fri, Aug 10, 6:04 PM · VDG, Plasma: KCM Redesign

Thu, Aug 9

rkflx committed R166:c15d63d78345: Fix more warnings uncovered by -Wswitch-enum (authored by rkflx).
Fix more warnings uncovered by -Wswitch-enum
Thu, Aug 9, 8:21 PM
rkflx requested review of D14716: Fix missing notification title found by -Wswitch-enum.
Thu, Aug 9, 8:10 PM
rkflx updated subscribers of D14588: Deduplicate default filename template code.

Anybody else from Spectacle wanna review this, since @ngraham is busy? (When I asked in the task whether to commit directly, the answer was a bit ambiguous…).

Thu, Aug 9, 1:13 PM
rkflx accepted D14202: Change default shortcut for "Zoom To Fit" action.

Thanks, works great. Unless @asturmlechner or @kmorwinski find anything wrong with this until Monday, let's land this to the frameworks branch.

Thu, Aug 9, 1:12 PM
rkflx accepted D14201: Set system default shortcut Ctrl+0 for "Actual Size" action.

@muhlenpfordt I'd say go ahead with landing (after updating the version). It would be good to still have some testing time before the next Frameworks release.

Thu, Aug 9, 1:12 PM · Frameworks
rkflx added a comment to D14700: Bump QT_MIN_VERSION and KF5_MIN_VERSION.

(I'll clean up the #ifdefs separately, mainly asking for general consensus for the version bump here.)

Thu, Aug 9, 8:22 AM
rkflx added a comment to T7927: Fonts “easy mode”.

In this mockup, is the bottom section dedicated to choose fonts differently than general but for specific elements, like a titlebar, window, buttons, etc?

Thu, Aug 9, 8:19 AM · VDG, Plasma: KCM Redesign
rkflx requested review of D14700: Bump QT_MIN_VERSION and KF5_MIN_VERSION.
Thu, Aug 9, 8:19 AM

Wed, Aug 8

rkflx committed R260:b883cb894c11: Undo custom support for ccache (authored by rkflx).
Undo custom support for ccache
Wed, Aug 8, 11:04 PM
rkflx added a comment to T7927: Fonts “easy mode”.

One more quick idea (have not really thought about it, no final wording etc.):

Wed, Aug 8, 9:07 PM · VDG, Plasma: KCM Redesign
rkflx added a comment to T7927: Fonts “easy mode”.

Yes, I had worried about that, but hadn't come up with a solution.

Wed, Aug 8, 8:48 PM · VDG, Plasma: KCM Redesign
rkflx added a comment to T7927: Fonts “easy mode”.

Have you thought about how to keep the simple and the advanced tabs in sync, in particular when the user chooses conflicting options in both of them at the same time?

Wed, Aug 8, 8:20 PM · VDG, Plasma: KCM Redesign
rkflx accepted D14662: KPropertiesDialog: switch to label in setFileNameReadOnly(true).
Wed, Aug 8, 7:55 AM · Frameworks

Tue, Aug 7

rkflx added a comment to D14578: Fix video controls volume slider not vertically centered sometimes.

It seems to come from a Phonon::AudioOutput::volumeChanged signal but no idea why this happens only sometimes.

Tue, Aug 7, 10:17 PM
rkflx committed R260:a8dcc6cade3c: Deduplicate some conditional branches in RasterImageView (authored by rkflx).
Deduplicate some conditional branches in RasterImageView
Tue, Aug 7, 10:14 PM
rkflx added a comment to D14662: KPropertiesDialog: switch to label in setFileNameReadOnly(true).

Thanks for the help. Works great, just one inline question about an edge case (everything else LGTM).

Tue, Aug 7, 5:25 PM · Frameworks
rkflx accepted D14512: Make dependency on KActivities optional.

Thanks, LGTM now.

Tue, Aug 7, 5:24 PM
rkflx added a comment to D7581: HiDPI Support for Gwenview.

how to handle the freedesktop.org thumbnail caching spec

Tue, Aug 7, 5:24 PM

Mon, Aug 6

rkflx committed R260:b630e152bb60: Remove no-ops when adjusting URLs (authored by rkflx).
Remove no-ops when adjusting URLs
Mon, Aug 6, 9:48 PM
rkflx added a comment to D13901: Touch support for Gwenview.

Are there some quirks we didn't account for when you git checkout a specific revision and build it?

Mon, Aug 6, 8:07 PM
rkflx added a comment to D14583: [MainWindow] Navigate through mouse forward/back buttons.

What fullscreen issue?

Mon, Aug 6, 8:05 PM
rkflx added a comment to D14610: Use KLineEdit for folder name if folder has write access, else use QLabel.

Hmm, well, for IconApplet's use case

Mon, Aug 6, 7:46 PM · Frameworks
rkflx requested changes to D14583: [MainWindow] Navigate through mouse forward/back buttons.

Thanks for the update. This leaves us with the Browse mode and Fullscreen issues.

Mon, Aug 6, 7:45 PM
rkflx added inline comments to D14512: Make dependency on KActivities optional.
Mon, Aug 6, 7:45 PM
rkflx updated the diff for D14588: Deduplicate default filename template code.
  • Use function instead of static const
Mon, Aug 6, 5:54 PM
rkflx updated subscribers of D14583: [MainWindow] Navigate through mouse forward/back buttons.

There are two more problem I noticed, this time in View mode:

  • Repeatedly clicking fast on one of the forward or backward buttons will only change the image for every second click.
  • Doing the same for videos will accidentally trigger fullscreen mode.
Mon, Aug 6, 5:53 PM
rkflx requested changes to D7581: HiDPI Support for Gwenview.

Yeah, a lot has happened since last year. After the finishing touches are done, this should be checked carefully again.

Mon, Aug 6, 5:53 PM

Sun, Aug 5

rkflx committed R260:4655089fc732: Remove extra break statements (authored by rkflx).
Remove extra break statements
Sun, Aug 5, 10:54 PM
rkflx added a comment to D14449: Modify device usage information.

+1 for using the same label but putting the information on more than one line like @rkflx suggests.

Sun, Aug 5, 10:42 PM · Frameworks
rkflx added a comment to T9250: Revive bug triaging days.
Sun, Aug 5, 10:42 PM
rkflx added a comment to D14449: Modify device usage information.

This comment has been deleted.

Sun, Aug 5, 3:03 PM · Frameworks
rkflx requested changes to D14449: Modify device usage information.

Thanks, looking better than before now. There are still some improvements you could make:

  • There is a superfluous space before the comma in the second line.
  • I'd prefer the bar to be a bit wider by default. However, it turns out there is a problem with my original suggestion (see inline comment).
  • The vertical spacing between the first and the second line is too big, it should be the same as for Size:. However, there should still be enough spacing so it also looks good with the Oxygen style. As far as I can see this is an issue with how Breeze renders the KCapacityBar, in particular the bounding rect contains unnecessary margins (+Ctrl-click on it in GammaRay and compare Breeze and Oxygen). Of course that's material for a patch in a different repo, but the "hole" in your current screenshot does not look good (the second line is closer to the bottom than to the first line, which is bad!), and it would be better to fix the problem there before landing the KIO patch.
Sun, Aug 5, 12:37 PM · Frameworks
rkflx added a comment to D14610: Use KLineEdit for folder name if folder has write access, else use QLabel.

KPropertiesDialog::setFileNameReadOnly
m_bFromTemplate

Sun, Aug 5, 12:37 PM · Frameworks
rkflx added a comment to D14449: Modify device usage information.

Let me plug my suggestion again:

Device capacity: ==----- 24% used of 94.4 Gib
                 22.5 GiB used, 71.9 GiB free

if this design is used , then the capacity bar would look elongated and stretched, which doesn't looks good.

Sun, Aug 5, 9:13 AM · Frameworks

Sat, Aug 4

rkflx added a comment to D14378: Remove custom icon selection for trash.
In D14378#303463, @pino wrote:

... that change the role I have in this review. Sorry, I will not do that. You have already all the data needed, including the fact that the patch is acked already. Unless you really care about the green tick, just to see that "everybody accepted it".

Sat, Aug 4, 10:52 PM · Dolphin
rkflx added a comment to D14610: Use KLineEdit for folder name if folder has write access, else use QLabel.

@shubham Thanks for helping out with T9297!

Sat, Aug 4, 10:52 PM · Frameworks
rkflx accepted D14578: Fix video controls volume slider not vertically centered sometimes.

Thanks for the updates, LGTM ;)

Sat, Aug 4, 10:51 PM
rkflx committed R260:faa2be4ad6bf: Remove disabled kipi include (authored by rkflx).
Remove disabled kipi include
Sat, Aug 4, 10:50 PM

Fri, Aug 3

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

To properly test whether your patch introduces a regression, you would have to compare Gwenview's behaviour with 2324a8681a44 against that with D13901.

Yep, that's exactly what I did. 2324a8681a44 does not have the bug.

Fri, Aug 3, 10:34 PM
rkflx added a comment to D14578: Fix video controls volume slider not vertically centered sometimes.

Initialize HudSlider for video controls after widget is setup

Fri, Aug 3, 10:34 PM
rkflx updated the task description for T9252: Improve default filename when saving.
Fri, Aug 3, 9:58 PM · Spectacle
rkflx added a revision to T9252: Improve default filename when saving: D14588: Deduplicate default filename template code.
Fri, Aug 3, 9:58 PM · Spectacle
rkflx requested review of D14588: Deduplicate default filename template code.
Fri, Aug 3, 9:58 PM
rkflx committed R260:2aa2a4e127f4: Remove superfluous Qt version check (authored by rkflx).
Remove superfluous Qt version check
Fri, Aug 3, 9:57 PM
rkflx accepted D14578: Fix video controls volume slider not vertically centered sometimes.

Nice! This fixes even the time slider which showed the same issue when trying to open a video with an unsupported codec.

Fri, Aug 3, 9:55 PM
rkflx added a comment to D7087: Add "Copy Info" button to the About System KCM.

Happy birthday D7087, you are now 1 year old.

Fri, Aug 3, 9:54 PM · Plasma
rkflx added a comment to D14583: [MainWindow] Navigate through mouse forward/back buttons.

Yeah, click in Browse window buttons are not handled, it's needed there ?

Fri, Aug 3, 5:30 PM
rkflx added a comment to D14583: [MainWindow] Navigate through mouse forward/back buttons.

It works everywhere, i use such a mice for years (more than 5) you miss a lot :)

Fri, Aug 3, 5:15 PM
rkflx requested changes to D14583: [MainWindow] Navigate through mouse forward/back buttons.

Okay, seems nobody around here has such a mouse anymore, but I think in View mode this is working fine.

Fri, Aug 3, 5:07 PM
rkflx added inline comments to D14583: [MainWindow] Navigate through mouse forward/back buttons.
Fri, Aug 3, 4:38 PM
rkflx added inline comments to D14583: [MainWindow] Navigate through mouse forward/back buttons.
Fri, Aug 3, 4:31 PM
rkflx requested changes to D14583: [MainWindow] Navigate through mouse forward/back buttons.

Thanks, I'll have to look for such a device. Meanwhile, I've got a couple of inline comments.

Fri, Aug 3, 4:24 PM
rkflx added a comment to D14583: [MainWindow] Navigate through mouse forward/back buttons.

Thanks for the patch.

Fri, Aug 3, 4:06 PM
rkflx added a comment to D13901: Touch support for Gwenview.

after
git checkout 2324a8681a44
and
arc patch D13901
I have found the same issue.

Fri, Aug 3, 10:43 AM

Thu, Aug 2

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

@steffenh, I'm afraid I've found a regression in your patch as compared to git master from the point where you branched (2324a8681a444cd1b9b441e481adb15c09f92837)

Thu, Aug 2, 9:32 PM
rkflx committed R166:c7a8e8cafe91: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
Thu, Aug 2, 11:50 AM
rkflx committed R260:330c6cfa9fae: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
Thu, Aug 2, 11:45 AM
rkflx committed R260:c61693ccff2b: Disable setImageData for dragging of unmodified images (authored by rkflx).
Disable setImageData for dragging of unmodified images
Thu, Aug 2, 11:42 AM
rkflx closed D14543: Disable setImageData for dragging of unmodified images.
Thu, Aug 2, 11:42 AM

Wed, Aug 1

rkflx committed R260:2d06e2a0efe1: Remove the last reimp markers remaining (authored by rkflx).
Remove the last reimp markers remaining
Wed, Aug 1, 9:44 PM
rkflx requested changes to D14512: Make dependency on KActivities optional.

Thanks for the patch.

Wed, Aug 1, 9:38 PM
rkflx requested review of D14543: Disable setImageData for dragging of unmodified images.
Wed, Aug 1, 9:37 PM
rkflx added a comment to D13249: Refine behaviour for dragging modified images to external applications.

Spectacle seems to have a similiar issue when dragging the screenshot preview to the desktop.

That's interesting, thanks for the hint! I assume you are running Spectacle 17.12, but since Spectacle 18.04 there is D10427, which actually removes setImageData to make it work for Chromium, and apparently also fixes the issue you are reporting when dragging to the desktop.

Correct - I used 17.12.3. Tested with 18.07.80 in a Neon VM does not show this problem (but drop on desktop does nothing).

Wed, Aug 1, 9:34 PM
rkflx accepted D9342: Make sure, a newly saved image is selected in ContextManager.

Let's hope for the best ;)

Wed, Aug 1, 9:34 PM
rkflx added a comment to D9342: Make sure, a newly saved image is selected in ContextManager.

Added modifications from P250 by rkflx

Wed, Aug 1, 4:01 PM
rkflx added a comment to D9342: Make sure, a newly saved image is selected in ContextManager.

Okay, here's my patch (don't laugh, I know it looks fishy):

1diff --git a/lib/contextmanager.cpp b/lib/contextmanager.cpp
2index b04e4026..2243ea99 100644
3--- a/lib/contextmanager.cpp
4+++ b/lib/contextmanager.cpp
5@@ -348,7 +348,6 @@ void ContextManager::selectUrlToSelect()
6 // and manually set current URL
7 d->mSelectionModel->clearSelection();
8 setCurrentUrl(d->mUrlToSelect);
9- d->mUrlToSelect.clear();
10 }
11 }
12
13diff --git a/lib/documentview/documentview.cpp b/lib/documentview/documentview.cpp
14index fa9d3f5b..ee8dd85e 100644
15--- a/lib/documentview/documentview.cpp
16+++ b/lib/documentview/documentview.cpp
17@@ -464,6 +464,10 @@ DocumentView::DocumentView(QGraphicsScene* scene)
18 d->setCurrentAdapter(new EmptyAdapter);
19
20 setAcceptDrops(true);
21+
22+ connect(DocumentFactory::instance(), &DocumentFactory::documentChanged, this, [this]() {
23+ d->updateCaption();
24+ });
25 }
26
27 DocumentView::~DocumentView()

Wed, Aug 1, 4:57 AM
rkflx updated the title for P250 P170 replacement from untitled to P170 replacement.
Wed, Aug 1, 4:55 AM

Tue, Jul 31

rkflx committed R241:9829d0ef9e74: Top-align labels in properties dialog (authored by rkflx).
Top-align labels in properties dialog
Tue, Jul 31, 5:55 PM
rkflx closed D14501: Top-align labels in properties dialog.
Tue, Jul 31, 5:55 PM · Frameworks
rkflx added a comment to D14501: Top-align labels in properties dialog.

Just realized this actually fixes a problem present in KDE 3 already (KDE 2 did not have the second line yet, as far as I could tell in a short but very interesting test).

Tue, Jul 31, 5:55 PM · Frameworks
rkflx triaged T9297: Polish file/folder properties dialog as Low priority.
Tue, Jul 31, 5:54 PM · Frameworks