Review Request: Fix wrongly always present dot behind bullet in preview of the Paragraph dialog
Pierre Stirnweiss
pstirnweiss at googlemail.com
Mon Mar 26 08:00:59 BST 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104411/#review11866
-----------------------------------------------------------
This preview will have to go rather soonish. What needs to be done here is to replace the whole preview with a thumbnail created by the KoStyleThumbnailer (in the textlayout lib).
That way, the preview is really using the same layout/drawing stuff as the shapes. Now it is all done specific to this preview.
- Pierre Stirnweiss
On March 26, 2012, 6:52 a.m., Friedrich W. H. Kossebau wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104411/
> -----------------------------------------------------------
>
> (Updated March 26, 2012, 6:52 a.m.)
>
>
> Review request for Calligra.
>
>
> Description
> -------
>
> The preview of the Paragraph settings dialog always shows a . behind the bullet item, also for non-numeric bullet types, if no prefix is set.
> In the document things seems fine, it seems just the dialog which behaves not well... bad, bad dialog... :)
>
> For RTL types there might be another issue, but not really related. And I have no reap experience with that. Just mentioning, in case the reviewer wants to do something about it now she is looking at the code ;)
>
> Okay to backport to 2.4?
>
>
> Diffs
> -----
>
> plugins/textshape/dialogs/ParagraphBulletsNumbers.cpp 7964a7d
>
> Diff: http://git.reviewboard.kde.org/r/104411/diff/
>
>
> Testing
> -------
>
> Tried various bullet settings, no problem seen.
>
>
> Thanks,
>
> Friedrich W. H. Kossebau
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120326/e531212c/attachment.htm>
More information about the calligra-devel
mailing list