----------------------------------------------------------- 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 > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
