In D9342#301811, @muhlenpfordt wrote:Added modifications from P250 by rkflx
- Queries
- All Stories
- Search
- Advanced Search
Feed Advanced Search
Advanced Search
Advanced Search
Aug 1 2018
Aug 1 2018
Okay, here's my patch (don't laugh, I know it looks fishy):
1 | diff --git a/lib/contextmanager.cpp b/lib/contextmanager.cpp |
---|---|
2 | index 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 | |
13 | diff --git a/lib/documentview/documentview.cpp b/lib/documentview/documentview.cpp |
14 | index 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() |
Jul 31 2018
Jul 31 2018
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).
Drag-and-drop to the Desktop still works
rkflx added a comment to D13249: Refine behaviour for dragging modified images to external applications.
Thanks for investigating!
Jul 30 2018
Jul 30 2018
I knew I came across this link before, and indeed, we have a bug about this: https://bugs.kde.org/show_bug.cgi?id=228934. Quote:
Hi, thanks for your contribution.
Thanks for the patch.
rkflx added a comment to D13249: Refine behaviour for dragging modified images to external applications.
Gwenview, we have a problem:
Short report on this issue: I worked pretty hard on this the last weekend and also the weeks before. Every time I think I got it, there is one minor detail breaking everything again. There must be a way to eliminate mDirListerFinished (and meanwhile I also suspect mUrlToSelect can and should go), however I still have not found it :/
Add shortcut for Red Eye Reduction tool
In D14449#300841, @dhaumann wrote:Device capacity: 94.4 Gib Device usage: ==------ 24% full (71.9 GiB free, 22.5 GiB used)Could you give this a try with screenshot?
In D14449#300327, @ngraham wrote:Here's another idea for how to express everything while remaining compact in the horizontal dimension:
Device capacity: ====---------- 24% full 22.5 GiB used, 71.9 GiB free 94.4 Gib total size
Jul 29 2018
Jul 29 2018
In D14449#300086, @ngraham wrote:Device capacity: 94.4 Gib Device usage: ==------ 24% full (71.9 GiB free, 22.5 GiB used)
Improve wording for red eye reduction
Awesome. I'll look at it soon (but not sure how much time I can spend during the week, so please bear with me…).
In D14407#299046, @rkflx wrote:Use "Reduce Red Eye" for now.
I guess "Fix Red Eye" or "Correct Red Eye" would also be fine, I cannot really make up my mind. @ngraham Let me know if you prefer one of those.
Jul 28 2018
Jul 28 2018
rkflx committed R260:4acdc4d2886c: Fix integer underflow warning in ImageMetaInfoModel (authored by rkflx).
Fix integer underflow warning in ImageMetaInfoModel
Jul 27 2018
Jul 27 2018
rkflx added projects to T9273: Announcement text for the Applications 18.08 release: Gwenview, Spectacle.
Checked this now more thoroughly. There are a couple of unrelated changes that sneaked in, but it's mostly fine I guess.
@lbeltrame fixed it for @mlaurent in 1cab13f0970f – Thanks.
rkflx committed R260:034a61ea83ac: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
Remove crosshair cursor from crop tool
In D14406#298917, @muhlenpfordt wrote:I tested a couple of situations in crop tool witch your first patch but never noticed the cross hair cursor until you pointed it out.
Use "Reduce Red Eye" for now.
Jul 26 2018
Jul 26 2018
@muhlenpfordt Did you get a notification about my last comment here? If not, perhaps you should consider adding yourself as a "Watcher" of the project…
This breaks the build, please fix: https://build.kde.org/job/Applications%20gwenview%20kf5-qt5%20FreeBSDQt5.10/63/console
I mainly opened this Diff to discuss about the wording of that tool in general. In particular in the sidebar most image operations use the imperative form, only Red Eye Reduction is different. This is relevant not only in the sidebar, but also in the Edit menu and for the accepting button (future patch ;).
I (and the reviewers for D14386) should have grepped for Qt::CrossCursor… Anyway, we catched it in time ;)
rkflx committed R260:e06cac87196d: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
rkflx committed R260:e97eda1df871: Fix crosshair cursor not showing in red eye reduction tool (authored by rkflx).
Fix crosshair cursor not showing in red eye reduction tool
Wow, I didn't even know Gwenview did that!
In D14324#298315, @muhlenpfordt wrote:Found the reason for this. :)
The previous mousePressEvent needs to be accepted somewhere (for zooming in AbstractImageView), otherwise the subsequent mouseReleaseEvent is not triggered.
Jul 25 2018
Jul 25 2018
In D13901#297753, @steffenh wrote:It sounds a bit strange that setting different keyboard shortcuts for the rotate actions will get it working. Is it possible that you have some kind of global gesture recognizer running in your desktop session, which detects that action and then sends the keyboard shortcut combination to Gwenview?
Yes, I know it is strange. I found this in OpenSuse Leap 15.0 and a fresh KDE Neon 5.13, no global gesture recognizer is running.I found this by accident, I was looking which event I get in the DocumentView::sceneEventFilter with a qDebug()<<event;, to my surprise I found this:
rkflx committed R260:619dcde267fd: Always set transformation mode when setting zoom in image scaler (authored by rkflx).
Always set transformation mode when setting zoom in image scaler
rkflx accepted D14252: Rework rectangle/arrow key handling to avoid "stuck edges" and 0x0px rectangle.
Thanks, can confirm that the issue with doubled resizing is fixed. Accepting for now, even though the code is not the most concise yet.
@steffenh Could you report how touchpad gestures are working for you in Qt's example app (see https://doc.qt.io/qt-5/qtwidgets-gestures-imagegestures-example.html)?
Off you go!
Jul 24 2018
Jul 24 2018
rkflx retitled D14252: Rework rectangle/arrow key handling to avoid "stuck edges" and 0x0px rectangle from Rework rectamgle/arrow key handling to avoid "stuck edges" and 0x0px rectangle to Rework rectangle/arrow key handling to avoid "stuck edges" and 0x0px rectangle.
rkflx requested changes to D14252: Rework rectangle/arrow key handling to avoid "stuck edges" and 0x0px rectangle.
Thanks, this fixes the issue of getting stuck, as well as the minimum size.
Haha, I worked on the same issue, but you catched an earlier time window to post the Diff. Good for you ;) That said, this way I also know exactly where I can break you patch (see inline comment, obviously was also broken in the KDE 4 version).
Thanks! Uses accept and ignore just like I suspected when suggesting this, and works very well for middle click.
rkflx committed R260:1ce062a458b1: Fix image smoothing sometimes not getting applied (authored by rkflx).
Fix image smoothing sometimes not getting applied
rkflx committed R260:07ff10d35b6d: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
Jul 23 2018
Jul 23 2018
LGTM, works great!
Thanks for your support ;)
In T9250#152470, @xyquadrat wrote:After your research it'd be better to pick another program, but I am not sure which choice would work out well. Do you have an idea for an alternative?
@ngraham Would this patch be something you could back?
In D14284#296439, @steffenh wrote:Thank you for the help, I have it working now.
Thanks for the initiative, Julian.
In D14284#296276, @steffenh wrote:I not sure, I have used KDevelop 5.2.3 to get the Gwenview source from git. I make a new branch from master to do my coding and use the run or the debug function from KDevelop to test my work. It should pick up my self-compiled library or not?
rkflx committed R166:c8b41805e427: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
Add license header to TextLabel.qml
rkflx added a comment to D11203: Prevent grouping/duplicating notifications and fix "Open" button issue.
In D11203#296151, @kapillamba4 wrote:In D11203#273352, @rkflx wrote:Fix "Open" button in screenshot notification not responding to click
I don't really understand how to reproduce the problem from your test plan. Could you add detailed steps to reproduce, so I can check how it works before and after your patch? Ideally in terms of Spectacle's command line arguments, which should be similar to the global shortcuts.
In the first screen recording you can see that open button does not work sometimes, it might be because of QTimer:singleShot
rkflx added a comment to D11203: Prevent grouping/duplicating notifications and fix "Open" button issue.
In D11203#296132, @kapillamba4 wrote:@rkflx i think someone else should take over
In D14284#296087, @steffenh wrote:display the active image name, if you have more than one image in the view mode
Do you mean that in Compare mode the window title should change when clicking on another image? For me this is already the case without your patch.
Interesting, in KDE Neon 5.13 (Plasma 5.13.2, KDE Framework 5.47.0, Qt 5.11.0) yes it works without my patch. In my working Linux (OpenSuse Leap15.0, Plasma 5.12.5, KDE Framework 5.45.0, Qt 5.9.4) is it doesn't work without my patch.
Bug 378061
Thanks for the review!
Interesting idea ;)
rkflx updated the title for P246 gwenview-startpage-browse from untitled to gwenview-startpage-browse.
rkflx updated the title for P245 gwenview-startpage-setcaption from untitled to gwenview-startpage-setcaption.
rkflx updated the title for P244 gwenview-startpage-refactor from untitled to gwenview-startpage-refactor.
Jul 22 2018
Jul 22 2018
rkflx committed R166:311749e9ba7f: Merge remote-tracking branch 'origin/Applications/18.08' (authored by rkflx).
Merge remote-tracking branch 'origin/Applications/18.08'
rkflx committed R166:e0b52fa258c9: Fix rendering of on-screen help texts for fractional scaling (authored by rkflx).
Fix rendering of on-screen help texts for fractional scaling
Jul 21 2018
Jul 21 2018
rkflx added a comment to D11203: Prevent grouping/duplicating notifications and fix "Open" button issue.
In D11203#273352, @rkflx wrote:It seems like your patch fixes multiple things, so it would be better if you could split your patch into separate Diffs too (see also https://community.kde.org/Infrastructure/Phabricator#.22Please_do_that_in_a_separate_commit.22).
Refactoring sneak peek (headed for master, will post a Diff once the bug fix landed), which still needs testing and a commit message: P243
rkflx updated the title for P243 Image smoothing refactoring from untitled to Image smoothing refactoring.
rkflx committed R260:db566133b6e6: Adapt comment to recent change in smoothing threshold (authored by rkflx).
Adapt comment to recent change in smoothing threshold
Jul 20 2018
Jul 20 2018
Add comment. There is no TODO, because I hope with D12626 we'll still be able to support Qt 5.6.
rkflx committed R166:9de3f7628ef8: Fix Clang's mismatched tags warning instead of disabling it (authored by rkflx).
Fix Clang's mismatched tags warning instead of disabling it
Jul 19 2018
Jul 19 2018
Thanks for the explanation.
Is there something wrong with our Git branch setup for KDE Applications? Today Scott, Peter and you all had problems with merging stable into master…
Thanks, commit it already ;)
In D14196#294948, @ngraham wrote:"while dragging selection box" is pretty common and well understood in English.