https://bugs.kde.org/show_bug.cgi?id=364822
Henrik Fehlauer <rk...@lab12.net> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Status|CONFIRMED |RESOLVED
Latest Commit| |https://commits.kde.org/gwe
| |nview/9b1714fc7b50be746f39e
| |835c8b3146c38a9ddd9
Version Fixed In| |17.12.3
--- Comment #3 from Henrik Fehlauer <rk...@lab12.net> ---
Git commit 9b1714fc7b50be746f39e835c8b3146c38a9ddd9 by Henrik Fehlauer.
Committed on 01/03/2018 at 23:27.
Pushed by rkflx into branch 'Applications/17.12'.
Enlarge small SVGs too
Summary:
For images smaller than the viewport at 100% zoom, Gwenview allows to
scale up the image to {nav Fit} the window if {nav Enlarge smaller images}
is checked in the configuration dialog. So far this only worked for
raster-based images, but not for SVGs.
`mEnlargeSmallerImages` is meant to be handled by the abstract base
class `AbstractImageView`, however when f00cef3c0 moved code over there
from `RasterImageView`, it was missed to also adapt `SvgImageView`. This
can be fixed by implementing the missing function.
In addition, a left-over `bool` is removed.
FIXED-IN: 17.12.3
Test Plan:
- Open `icons/hisc-apps-gwenview.svgz`, press {nav Fit}: Small icon shown.
- Check {nav Enlarge smaller images}, press {nav Fit} again: Icon scaled up to
fit window.
- Enlarging raster-based images still works fine.
Reviewers: #gwenview, huoni, muhlenpfordt, ngraham
Reviewed By: #gwenview, huoni, muhlenpfordt, ngraham
Subscribers: ngraham, muhlenpfordt, huoni
Differential Revision: https://phabricator.kde.org/D10926
M +0 -2 lib/documentview/rasterimageview.cpp
M +6 -0 lib/documentview/svgviewadapter.cpp
M +2 -0 lib/documentview/svgviewadapter.h
https://commits.kde.org/gwenview/9b1714fc7b50be746f39e835c8b3146c38a9ddd9
--
You are receiving this mail because:
You are watching all bug changes.
