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


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

- Casper


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
> 
>

_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to