D14103: Add mouse shortcut for Fill and adapt docbook

[Henrik Fehlauer]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-doc-english/attachments/20180715/2bcaa629/attachment.html>