Review Request 108420: "Fit Text Width" zoom mode

Dmitry Kazakov dimula73 at gmail.com
Fri Jan 18 15:07:47 GMT 2013


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



libs/widgets/KoZoomController.cpp
<http://git.reviewboard.kde.org/r/108420/#comment19597>

    I have such a feeling that this explicit signal emission can be removed, because all the code paths above end up with recenterPreferred() or setPreferredCenter(), which should emit the signals as well. So we probably get duplicated signals here.
    
    Could you check whether it works without these lines?
    
    The rest is ok to commit.
    
    


- Dmitry Kazakov


On Jan. 15, 2013, 5:04 a.m., C. Boemann wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108420/
> -----------------------------------------------------------
> 
> (Updated Jan. 15, 2013, 5:04 a.m.)
> 
> 
> Review request for Calligra and Inge Wallin.
> 
> 
> Description
> -------
> 
>     Introduce a "Fit Text Width" zoom mode
>     
>     This also changes how the other "fit" modes work: We now zoom in to the fit the
>     max pagesize of the currently shown pages. Previously we zoomed in to the max
>     pagesize of the entire document.
>     
>     The page counter in the lower left of Words' window now show the scroll position
>     and not the cursorposition. Previously it was a weird mix
>     
>     The page counter also now shows ranges so that if pages 3,4,5 are shown, the counter
>     says "3-5 of 47"
> 
> 
> Diffs
> -----
> 
>   libs/widgets/KoZoomAction.cpp 589b4c7 
>   libs/widgets/KoZoomController.h 864abb4 
>   libs/widgets/KoZoomController.cpp 7cb6f06 
>   libs/widgets/KoZoomMode.h cfaeed8 
>   libs/widgets/KoZoomMode.cpp 81ffdf4 
>   words/part/KWStatusBar.cpp f3de247 
>   words/part/KWView.h 253b8021 
>   words/part/KWView.cpp 988dba4 
> 
> Diff: http://git.reviewboard.kde.org/r/108420/diff/
> 
> 
> Testing
> -------
> 
> only in words, and since it touches libs too I'd like some feedback
> 
> 
> Thanks,
> 
> C. Boemann
> 
>

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


More information about the calligra-devel mailing list