Review Request 120733: Pass more data between layout and RootAreaProvider

Pierre Ducroquet pinaraf at pinaraf.info
Tue Oct 28 08:29:21 GMT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120733/
-----------------------------------------------------------

(Updated Oct. 28, 2014, 8:29 a.m.)


Review request for Calligra and Camilla Boemann.


Changes
-------

This new version moves more logic to the provider.

Following discussions with boemann, the releaseAllAfter mechanism is still used, but a TODO is left for the future in the provider if we find a simple way to insert pages in the middle of the page flow without too many side effects. Since more logic has been moved to the provider, the layout code got simplified by removing a double loop doing almost exactly the same things and depending more on the provider instead.


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 (updated)
-----

  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/KWDocument.cpp e287829329f5026e455081978e8f345c4883617d 
  words/part/KWRootAreaProvider.h 8fb9b5e379a0561fceb7ba32935084df46ce6fd9 
  words/part/KWRootAreaProvider.cpp ce039219c1a044b13ae81420832eb894346b88db 

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/20141028/3e8349d6/attachment.htm>


More information about the calligra-devel mailing list