-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119894/#review65121
-----------------------------------------------------------
So we had a loop that set d->zoomFactor repeatedly, I didn't know that, I guess
the last iteration was the one that won :P
I've tried your patch on the document from the bug and I'm noticing this issue:
if you choose, say, "fit width" on the first page, go to the second page and
zoom out, you actually zoom in. Is it intended behavior? I would expect zoom
out to always reduce the page size on the screen.
What about moving the "use the current page as reference" thing when the user
actually changes zoom, so that zoom in/out always take the real current page as
reference? Something like this:
@@ -3660,6 +3660,10 @@ void PageView::updateZoom( ZoomMode newZoomMode )
zoomValue[13] = zoomFactorFitWidth;
zoomValue[14] = zoomFactorFitPage;
qSort(zoomValue.begin(), zoomValue.end());
+
+ if ( d->items.count() > 0 )
+ newFactor = d->items[ qMax( 0, (int)d->document->currentPage()
) ]->zoomFactor();
+
QVector<float>::iterator i;
if ( newZoomMode == ZoomOut )
{
- Fabio D'Urso
On Aug. 21, 2014, 10:08 p.m., Albert Astals Cid wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119894/
> -----------------------------------------------------------
>
> (Updated Aug. 21, 2014, 10:08 p.m.)
>
>
> Review request for Okular, Markus Trippelsdorf and Tingnan Zhang.
>
>
> Bugs: 335819
> http://bugs.kde.org/show_bug.cgi?id=335819
>
>
> Repository: okular
>
>
> Description
> -------
>
> In the wild world where each page is of different size, only use the current
> page to update the zoom factor in updateItemSize
>
>
> Diffs
> -----
>
> ui/pageview.cpp 8ceafae
>
> Diff: https://git.reviewboard.kde.org/r/119894/diff/
>
>
> Testing
> -------
>
> Works for me with https://bugs.kde.org/attachment.cgi?id=88318
>
>
> Thanks,
>
> Albert Astals Cid
>
>
_______________________________________________
Okular-devel mailing list
Okular-devel@kde.org
https://mail.kde.org/mailman/listinfo/okular-devel
