[Okular-devel] Review Request 112370: BugFix for bug 323434/323435

[Albert Astals Cid]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112370/#review40818
-----------------------------------------------------------



ui/pageview.cpp
<http://git.reviewboard.kde.org/r/112370/#comment29999>

    Sorry for tha late reply, been on holiday.
    
    Since incFactor and decFactor are only used inside some of the cases can you please move their definition insdei the case? Also make them const, and also we don't need modFactor out.
    
    


- Albert Astals Cid


On Aug. 30, 2013, 2:26 p.m., Tingnan Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112370/
> -----------------------------------------------------------
> 
> (Updated Aug. 30, 2013, 2:26 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Description
> -------
> 
> BugFix 323434/323435. Zoom factor now will be properly rounded to those "interval values" like 140%, 250%, etc, when using zoom in and out feature. 
> 
> 
> Diffs
> -----
> 
>   ui/pageview.cpp 0d6c567d836340555b3101b58178a9247959543a 
> 
> Diff: http://git.reviewboard.kde.org/r/112370/diff/
> 
> 
> Testing
> -------
> 
> done on local machine
> 
> 
> Thanks,
> 
> Tingnan Zhang
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20130925/f2f5709c/attachment.html>