rkflx added a comment.
In D14103#292346 <https://phabricator.kde.org/D14103#292346>, @muhlenpfordt wrote: > Sadly the centering to mouse position when zooming to 100% gets lost with our combination. Interesting. Is this really working for you without the patches? For me, even middle-clicking with a KDE4-based Gwenview centers instead of respecting the mouse position. > Should I add this to `DocumentView::toggleZoomTo...()`? It will work for both shortcut (maybe surprising for users?) and click. > Or add an argument to pass the mouse position only for click events? I think it would be great to get this working for both clicking and the keyboard shortcuts, see my comment in D14093#291581 <https://phabricator.kde.org/D14093#291581>. (I'm not sure how well this will work for the shortcut case when the cursor is outside the viewport. If it is confusing maybe in that case there could be a fallback to the center, but let's try without it first.) REPOSITORY R260 Gwenview REVISION DETAIL https://phabricator.kde.org/D14103 To: rkflx, #gwenview Cc: muhlenpfordt, kde-doc-english, skadinna
