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

[C. Boemann]

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


Hi suresh. Any prospect of you working on this to fix the issues raised?

- C. Boemann


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 C. Boemann.
> 
> 
> Description
> -------
> 
> 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/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 Chande
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120107/fc37f9b3/attachment.htm>