Review Request 116662: Navigator docker for Words

Denis Kuplyakov dener.kup at gmail.com
Tue Mar 11 12:06:53 GMT 2014


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

(Updated March 11, 2014, 12:06 p.m.)


Review request for Calligra, Inge Wallin and Dan Leinir Turthra Jensen.


Changes
-------

Fixed latest issues.


Repository: calligra


Description
-------

That's the navigator docker for Words. It allow to quickly navigate through document by jumping to sections headers.
Navigator looks through all paragraphs with outline property, and stores them in QTreeView allowing fast jump with single click.

I want some response about UI:
1) maybe make the rows bigger?

and about the implementation:
2) how can we track structure changes more effective to avoid complete rebuilding of tree?
3) view is flickering when updating..


Diffs (updated)
-----

  words/part/CMakeLists.txt e6e39e8 
  words/part/KWFactory.cpp 14dfbed 
  words/part/dockers/KWNavigationDocker.h PRE-CREATION 
  words/part/dockers/KWNavigationDocker.cpp PRE-CREATION 
  words/part/dockers/KWNavigationDockerFactory.h PRE-CREATION 
  words/part/dockers/KWNavigationDockerFactory.cpp PRE-CREATION 
  words/part/dockers/KWNavigationWidget.h PRE-CREATION 
  words/part/dockers/KWNavigationWidget.cpp PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/116662/diff/


Testing
-------

Have tested it on document, containing 3 pages with many headers of different levels.


File Attachments
----------------

Test file
  https://git.reviewboard.kde.org/media/uploaded/files/2014/03/08/4147a7ee-bb9b-4ff9-b809-fba7c2eb1b16__test.odt


Thanks,

Denis Kuplyakov

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


More information about the calligra-devel mailing list