Review Request: Fix for Reducing the various hardcoded zoomlevel based on usability feedback from Anna

[C. Boemann]

No I'm not talking about removing anything from the bottom. I'm saying your 
patch currently removes some functionality there, which I'm not sure we want 
removed.

Please try and click on the number in the bottom right corner (without your 
patch)

best regards
Casper

On Tuesday 05 April 2011 00:15:39 Suresh Chande wrote:
> > On April 4, 2011, 6:31 a.m., Casper Boemann wrote:
> > > I think the 100% should staty. That is what Anna-lisa said too
> > > 
> > > Also I'm not either for or against it, but the options in the widget in
> > > the bottom corner is removed too with this patch
> 
> by "but the options in the widget in the bottom corner is removed too with
> this patch" I guess you are talking about the slider for zooming, right ?.
> There was a discussion to add the slider into the zoom. i am not entirely
> comfortable as this is not totally according the UI Experience where a
> slider exists inside a drop-down menu.
> 
> 
> - Suresh
> 
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101022/#review2372
> -----------------------------------------------------------
> 
> On April 3, 2011, 9:12 p.m., Suresh Chande wrote:
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > http://git.reviewboard.kde.org/r/101022/
> > -----------------------------------------------------------
> > 
> > (Updated April 3, 2011, 9:12 p.m.)
> > 
> > 
> > Review request for Calligra, Marijn Kruisselbrink, Thorsten Zachmann, and
> > Casper Boemann.
> > 
> > 
> > Summary
> > -------
> > 
> > Ok Here comes my first fix..
> > 
> > Here is my fix I made to the zoom levels based on discussion and
> > agreement at the Calligra sprint that we do not want to have 33, 57 and
> > 127 etc % zoom levels . This fix removes the different zoom levels but
> > leaves fit to width and fit to page.
> > 
> > See testing done, Any suggestion how this should be it be fixed for
> > Tables ?
> > 
> > 
> > Diffs
> > -----
> > 
> >   libs/widgets/KoZoomAction.cpp 488652c
> > 
> > Diff: http://git.reviewboard.kde.org/r/101022/diff
> > 
> > 
> > Testing
> > -------
> > 
> > This was tested in the Words and Stage application and it functions
> > according to expected outcome. I tested the Tables it seems as Tables
> > does not have Fit to Width or Fit to Page the menu is currently empty.
> > 
> > 
> > Thanks,
> > 
> > Suresh