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



plugins/textshape/TextTool.cpp
<http://git.reviewboard.kde.org/r/105198/#comment11518>

    this line should be deleted



plugins/textshape/TextTool.cpp
<http://git.reviewboard.kde.org/r/105198/#comment11517>

    shouldn't you do the same for increase if >9



plugins/textshape/TextTool.cpp
<http://git.reviewboard.kde.org/r/105198/#comment11516>

    space



libs/textlayout/ListItemsHelper.h
<http://git.reviewboard.kde.org/r/105198/#comment11519>

    wow it seems we have a lot of duplication here with 
libs/odf/KoOdfNumberDefinition  ::formattedNumber


- C. Boemann


On June 10, 2012, 6:08 p.m., Gopalakrishna Bhat wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105198/
> -----------------------------------------------------------
> 
> (Updated June 10, 2012, 6:08 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> This patch fixes bugs 260083 and 298063 by increasing the display level of 
> the list when the list level is changed either by pressing tab or by using 
> the "Increase Indent" button.
> 
> 
> Diffs
> -----
> 
>   libs/kotext/commands/ChangeListCommand.cpp ea180fb 
>   libs/kotext/styles/KoListLevelProperties.cpp 12085e1 
>   libs/textlayout/ListItemsHelper.h c021b68 
>   libs/textlayout/ListItemsHelper.cpp d4e409c 
>   plugins/textshape/TextTool.cpp dff1fe1 
>   plugins/textshape/commands/ChangeListLevelCommand.cpp fdb5833 
> 
> Diff: http://git.reviewboard.kde.org/r/105198/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gopalakrishna Bhat
> 
>

_______________________________________________
calligra-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/calligra-devel

Reply via email to