Review Request 120733: Pass more data between layout and RootAreaProvider
Pierre Ducroquet
pinaraf at pinaraf.info
Fri Oct 24 08:34:10 BST 2014
> On Oct. 23, 2014, 7:28 a.m., Camilla Boemann wrote:
> > words/part/KWRootAreaProvider.cpp, line 168
> > <https://git.reviewboard.kde.org/r/120733/diff/1/?file=321689#file321689line168>
> >
> > i guess we should insert a page instead of appending (thus avoiding the need to release all after) if it mismatches the next available page in the current list of pages
> >
> > possibly this could be even smarter
>
> Pierre Ducroquet wrote:
> I partially agree, I'll try that way nevertheless, but I fear it's going to have much more side effects.
After a few hours fooling around the code, it seems the insertPage call does not work. It will require quite some patching, perhaps some important refactoring.
Inserting a page also implies moving every frame/shape downwards, recalculating every page position (right/left)… This is going to be a far more complex and riskier patch.
- Pierre
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120733/#review68957
-----------------------------------------------------------
On Oct. 23, 2014, 7:15 a.m., Pierre Ducroquet wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120733/
> -----------------------------------------------------------
>
> (Updated Oct. 23, 2014, 7:15 a.m.)
>
>
> Review request for Calligra and Camilla Boemann.
>
>
> Repository: calligra
>
>
> Description
> -------
>
> This patch refactors a bit the KoTextLayoutRootAreaProvider.
> It extends the provideNext call by adding a constraints structure.
> This structure contains so far the masterPageName and the page
> number of the requested new rootArea.
> This way, the provider don't have to sneak in layout specific data.
> It also adds a way for KoTextLayoutRootAreaProvider to help the
> layout and fix inserting a new page (by adding lines to a paragraph
> for instance) before a page break changing the page style.
>
> Some more explanations regarding this bug :
> - Page 1, style AAA
> * page break, requesting style BBB
> - Page 2, style BBB
>
> If you add more data to the page 1, then dynamically the layout should
> do :
> - Page 1, style AAA
> - Page 2, style AAA
> * page break, requesting style BBB
> - Page 3, style BBB
>
> But the layout did recycle the page 2, leading to :
> - Page 1, style AAA
> - Page 2, style BBB
> * page break, requesting style BBB
> - Page 3, style BBB
>
> Reloading the document leads to a correct layout.
>
>
> Diffs
> -----
>
> libs/textlayout/KoTextDocumentLayout.cpp 805f11d1c4aaf90a1a2a654053889e06800193de
> libs/textlayout/KoTextLayoutRootAreaProvider.h 23b91bc519761eef2861fc6fc4546294271389cc
> libs/textlayout/tests/MockRootAreaProvider.h 23056384fb55918ae360ad062d5dfd13ee7c4837
> libs/textlayout/tests/MockRootAreaProvider.cpp 0402528998c26cfb586f297abe7c879459adda50
> plugins/textshape/SimpleRootAreaProvider.h afada40af1ea3341a820dc0d275a2db5bbb5a3f8
> plugins/textshape/SimpleRootAreaProvider.cpp 42df2ecd30aa744780642618556a7013a9ff27da
> words/part/KWRootAreaProvider.h 8fb9b5e379a0561fceb7ba32935084df46ce6fd9
> words/part/KWRootAreaProvider.cpp 62b6fb6fba4dea8d9f68b0b3c1afb7324c50d502
>
> Diff: https://git.reviewboard.kde.org/r/120733/diff/
>
>
> Testing
> -------
>
> A simple document with page styles (changing the background color to help speed up the development) now doesn't exhibit the wrong layout behaviour.
>
>
> Thanks,
>
> Pierre Ducroquet
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141024/8cf5fb4e/attachment.htm>
More information about the calligra-devel
mailing list