muhlenpfordt added a comment.

  Code looks good to me.
  Sadly the centering to mouse position when zooming to 100% gets lost with our 
combination.
  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?

REPOSITORY
  R260 Gwenview

REVISION DETAIL
  https://phabricator.kde.org/D14103

To: rkflx, #gwenview
Cc: muhlenpfordt, kde-doc-english, skadinna

Reply via email to