Review Request: This patch supports the navigation of the text doc in CA according to user's wish

Sujith Haridasan sujith.h at gmail.com
Tue Feb 28 03:10:54 GMT 2012


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

(Updated Feb. 28, 2012, 3:10 a.m.)


Review request for Calligra and Shantanu Tushar Jha.


Changes
-------

Updated the new diff file with the following changes:
1) Removed the extra white spaces 
2) Changed the function name to initializePageIcons
3) Changed the copyright to 2012.


Description
-------

This patch allows the user to navigate through the pages according the wish. The "page + Number" format will be displayed on the left side of the CA Text doc. Clicking on them the user is expected to reach the top portion of that page selected.


Diffs (updated)
-----

  active/qml/TextDocumentLeftToolbar.qml PRE-CREATION 
  active/src/CATextDocumentHandler.h 8f592f3 
  active/src/CATextDocumentHandler.cpp 01bfeb0 

Diff: http://git.reviewboard.kde.org/r/104092/diff/


Testing
-------

Testing done:

1) The total number of pages displayed on the left portion is tested and it displays correctly.
2) When clicked on a particular page, the user doesn't reaches the exact location. Couldn't figure out
   the exact issue. Any suggestions?


Thanks,

Sujith Haridasan

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


More information about the calligra-devel mailing list