Hello Tim,
Thanks for your patch. :-)
I didn't test it yet, but just my nitpick: could you please remove the
#include rtl/math.hxx
which I added to make the build success ?
It seems to be unused now.
And FYI, Jan Holesovsky changed the zoom factor to 2^(1/4).
Hi Korrawit,
thanks for the information, I have attached a new patch and also 25% is
enforced like Stefan requested.
Regards
Tim
On Tuesday 24 January 2012 12:13:20 Korrawit Pruegsanusak wrote:
Hello Tim,
Thanks for your patch. :-)
I didn't test it yet, but just my nitpick: could you
Hi Stefan,
thanks for testing.
If 25% is an important number the function enforceStep could be used, if my
patch gets accepted.
But if other/more specific numbers are also important we really should consider
using static/preprocessed zooming levels instead like the Adobe Reader seems to
have.
Hi Tim,
finally got around to compiling with your patch in and testing the
whole thing a bit and think I like the experience. Although, I would
still love if you could make sure that we in all cases reach exactly
25% zoom at some point (not just 24 and 26).
Thanks for your work.
Astron.
Hi,
this patch does round zoom values beginning with 50 to a multiple of 5,
with 100 to one
of 10, with 500 to one of 50 and with 1000 to one of 100.
The step 100 is enforced to have one fixed point.
The zoom factor was also changed from 1.2 to 1.1.
Regards
Tim
--
SUSE LINUX Products GmbH,