----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109746/ -----------------------------------------------------------
(Updated March 30, 2013, 6:11 p.m.) Review request for Calligra. Changes ------- Improvements in the patch (primary) : 1. No warning boxes. The user is not allowed to commit mistakes. All notifications are done inline. 2. insertText() was extended to accommodate hyperlink insertion. Description ------- This patch adds the following features : 1. Inserting hyperlinks -- User has a choice of inserting a link by specifying the target and the link text. Additionally, the user may supply values for target and rel attributes using a drop down list. -- The user may fetch the title from the web page itself (contents of the "title" tag ). This feature also handles URL redirects. For example, ieee.com is finally redirected to http://www.ieee.org/index.html. This feature will especially help in cases when a user has a list of links referred to and a list of references has to be created. Just copy/pasting the URL, clicking fetch and then insert will do the job. 2. Linking to bookmarks -- A user can specify a bookmark name and the link text. To help the user with inserting bookmark, an auto completer is used. This becomes helpful when the bookmarks have been given a name that are related to the context. (lastpagefirstpara or conclusions). 3. Adding Bookmark using a labeled widget ( similar to the way footnote and endnote labels are entered ) Diffs (updated) ----- libs/kotext/KoTextEditor.h f96ab70 libs/kotext/KoTextEditor.cpp 07bcad8 plugins/textshape/CMakeLists.txt b2bf9ae plugins/textshape/ReferencesTool.h 21caea8 plugins/textshape/ReferencesTool.cpp 46bf06f plugins/textshape/dialogs/LinkInsertionDialog.h PRE-CREATION plugins/textshape/dialogs/LinkInsertionDialog.cpp PRE-CREATION plugins/textshape/dialogs/LinkInsertionDialog.ui PRE-CREATION plugins/textshape/dialogs/ManageBookmark.ui PRE-CREATION plugins/textshape/dialogs/ManageBookmarkDialog.h PRE-CREATION plugins/textshape/dialogs/ManageBookmarkDialog.cpp PRE-CREATION plugins/textshape/dialogs/SimpleCitationBibliographyWidget.ui 55699bb plugins/textshape/dialogs/SimpleLinksWidget.h PRE-CREATION plugins/textshape/dialogs/SimpleLinksWidget.cpp PRE-CREATION plugins/textshape/dialogs/SimpleLinksWidget.ui PRE-CREATION plugins/textshape/dialogs/SimpleTableOfContentsWidget.h adfd010 words/part/KWView.cpp 8639acd Diff: http://git.reviewboard.kde.org/r/109746/diff/ Testing ------- I have tested the following components : 1 Inserting Hyperlinks 1.1 Simplest case (works) 1.2 Fetch title from URL (works) 1.3 Fetch title from URL which has re directions (works) 1.4 specify a URL without a scheme (works, appending an http:// to all the schemeless urls) 1.5 fetching from an non existing URL (Time out occurs and notifies user about the same) 2 Adding Bookmarks 2.1 Simplest case (works) 2.2 Adding a duplicate bookmark i.e. using a name that has been used before(User is notified and bookmark addition is aborted) 3 Adding link to a bookmark 3.1 Simplest case (works) 3.2 Adding a link to a non existing bookmark (works,User is notified about non existence of the bookmark) 4 Manage bookmarks 4.1 user clicks on manage bookmarks (works, Copy pasted the existing implementation. Looks like delete bookmark is broken.) Thanks, Aman Madaan
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel