[prev in list] [next in list] [prev in thread] [next in thread]
List: calligra-devel
Subject: Re: Review Request 109746: Expansion of the References section by adding
From: "C. Boemann" <cbr () boemann ! dk>
Date: 2013-03-28 22:02:46
Message-ID: 20130328220246.13312.31180 () vidsolbach ! de
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109746/#review30045
-----------------------------------------------------------
the weblink and bookmark link widgets should just be inside the LinkInsertionDialog. \
Having 3x3 files for this just makes it more convoluted than it need to be.
qt esigner is capable of creating subwidgets within a tab widget, and you can easily \
refer to them from code.
plugins/textshape/ReferencesTool.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22388>
this is still a duplication of the action just above - a typo i believe
plugins/textshape/ReferencesTool.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22390>
{ for methods should be on a line of it's own
plugins/textshape/ReferencesTool.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22389>
indentation please
plugins/textshape/ReferencesTool.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22391>
{ for methods should be on a line of it's own
plugins/textshape/ReferencesTool.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22392>
too much indentation
plugins/textshape/dialogs/SimpleCitationBibliographyWidget.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22393>
no no, this is wrong - please don't :)
plugins/textshape/dialogs/SimpleLinksWidget.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22394>
this method should go completely
plugins/textshape/dialogs/SimpleLinksWidget.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22395>
this method should go completely
plugins/textshape/dialogs/WeblinkInsertionWidget.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22396>
indentation
plugins/textshape/dialogs/WeblinkInsertionWidget.cpp
<http://git.reviewboard.kde.org/r/109746/#comment22397>
indentation
- C. Boemann
On March 28, 2013, 8:42 p.m., Aman Madaan wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109746/
> -----------------------------------------------------------
>
> (Updated March 28, 2013, 8:42 p.m.)
>
>
> Review request for Calligra.
>
>
> 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
> -----
>
> plugins/textshape/CMakeLists.txt b2bf9ae
> plugins/textshape/ReferencesTool.h 21caea8
> plugins/textshape/ReferencesTool.cpp 46bf06f
> plugins/textshape/dialogs/BookmarkLinkInsertionWidget.h PRE-CREATION
> plugins/textshape/dialogs/BookmarkLinkInsertionWidget.cpp PRE-CREATION
> plugins/textshape/dialogs/BookmarkLinkInsertionWidget.ui PRE-CREATION
> plugins/textshape/dialogs/LinkInsertionDialog.h PRE-CREATION
> plugins/textshape/dialogs/LinkInsertionDialog.cpp PRE-CREATION
> plugins/textshape/dialogs/LinkInsertionDialog.ui PRE-CREATION
> plugins/textshape/dialogs/SelectBookmark.ui PRE-CREATION
> plugins/textshape/dialogs/SelectBookmarkDialog.h PRE-CREATION
> plugins/textshape/dialogs/SelectBookmarkDialog.cpp PRE-CREATION
> plugins/textshape/dialogs/SimpleBookmarkWidget.h PRE-CREATION
> plugins/textshape/dialogs/SimpleBookmarkWidget.cpp PRE-CREATION
> plugins/textshape/dialogs/SimpleBookmarkWidget.ui PRE-CREATION
> plugins/textshape/dialogs/SimpleCitationBibliographyWidget.cpp 2d1c1de
> 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
> plugins/textshape/dialogs/WeblinkInsertionWidget.h PRE-CREATION
> plugins/textshape/dialogs/WeblinkInsertionWidget.cpp PRE-CREATION
> plugins/textshape/dialogs/WeblinkInsertionWidget.ui PRE-CREATION
> 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
>
>
[Attachment #5 (text/html)]
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;"> <tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/109746/">http://git.reviewboard.kde.org/r/109746/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: \
-pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">the weblink and bookmark \
link widgets should just be inside the LinkInsertionDialog. Having 3x3 files for this \
just makes it more convoluted than it need to be.
qt esigner is capable of creating subwidgets within a tab widget, and you can easily \
refer to them from code.</pre> <br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125885#file125885line164" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/ReferencesTool.cpp</a> <span style="font-weight: \
normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">void \
ReferencesTool::createActions()</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">161</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="n">action</span> <span class="o">=</span> <span class="k">new</span> <span \
class="n">KAction</span><span class="p">(</span><span class="n">i18n</span><span \
class="p">(</span><span class="s">"Insert Custom Bibliography1"</span><span \
class="p">),</span> <span class="k">this</span><span class="p">);</span></pre></td> \
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">this is \
still a duplication of the action just above - a typo i believe</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125885#file125885line373" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/ReferencesTool.cpp</a> <span style="font-weight: \
normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">370</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="kt">void</span> <span class="n">ReferencesTool</span><span \
class="o">::</span><span class="n">insertLink</span><span class="p">()</span> <span \
class="p">{</span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">{ for \
methods should be on a line of it's own</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125885#file125885line374" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/ReferencesTool.cpp</a> <span style="font-weight: \
normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">371</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="k">new</span> <span class="n">LinkInsertionDialog</span><span \
class="p">(</span><span class="n">textEditor</span><span class="p">(),</span> <span \
class="n">m_slw</span><span class="p">);</span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">indentation \
please</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125885#file125885line377" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/ReferencesTool.cpp</a> <span style="font-weight: \
normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">374</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="kt">void</span> <span class="n">ReferencesTool</span><span \
class="o">::</span><span class="n">insertBookmark</span><span class="p">(</span><span \
class="n">QString</span> <span class="n">bookMarkName</span><span class="p">)</span> \
<span class="p">{</span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">{ for \
methods should be on a line of it's own</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125885#file125885line390" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/ReferencesTool.cpp</a> <span style="font-weight: \
normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">387</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="p">}</span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">too much \
indentation</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125898#file125898line106" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/dialogs/SimpleCitationBibliographyWidget.cpp</a> <span \
style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">void \
SimpleCitationBibliographyWidget::prepareTemplateMenu()</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2">106</font></th> <td bgcolor="#fdfebc" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="kt">void</span> \
<span class="n">SimpleCitationBibliographyWidget</span><span class="o">::</span><span \
class="n">insertCustomBibliography</span><span class="p">()</span></pre></td> <th \
bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">106</font></th> <td bgcolor="#fdfebc" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="kt">void</span> <span class="n">SimpleCitationBibliographyWidget</span><span \
class="o">::</span><span class="n">insertCustomBibliography</span><span \
class="p">()</span><span class="hl"> </span><span class="p"><span \
class="hl">{</span></span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">no no, this \
is wrong - please don't :)</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125901#file125901line106" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/dialogs/SimpleLinksWidget.cpp</a> <span \
style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">106</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="kt">void</span> <span class="n">SimpleLinksWidget</span><span \
class="o">::</span><span class="n">deleteBookmark</span><span class="p">(</span><span \
class="k">const</span> <span class="n">QString</span> <span \
class="o">&</span><span class="n">name</span><span class="p">)</span></pre></td> \
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">this method \
should go completely</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125901#file125901line136" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/dialogs/SimpleLinksWidget.cpp</a> <span \
style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">136</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span \
class="kt">void</span> <span class="n">SimpleLinksWidget</span><span \
class="o">::</span><span class="n">applyTemplate</span><span class="p">(</span><span \
class="kt">int</span> <span class="p">){</span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">this method \
should go completely</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125905#file125905line74" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/dialogs/WeblinkInsertionWidget.cpp</a> <span \
style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">74</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="n">displayMessageBox</span><span class="p">(</span><span \
class="s">"Fetching title is taking too long"</span><span \
class="p">,</span> <span class="s">"Please check the url and try \
again"</span><span class="p">);</span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">indentation</pre> </div>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; \
border-collapse: collapse; margin: 2px padding: 2px;"> <thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; \
font-size: 9pt; padding: 4px 8px; text-align: left;"> <a \
href="http://git.reviewboard.kde.org/r/109746/diff/2/?file=125905#file125905line88" \
style="color: black; font-weight: bold; text-decoration: \
underline;">plugins/textshape/dialogs/WeblinkInsertionWidget.cpp</a> <span \
style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" \
align="right"><font size="2"></font></th> <td bgcolor="#c5ffc4" width="50%"><pre \
style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td> <th \
bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid \
#C0C0C0;" align="right"><font size="2">88</font></th> <td bgcolor="#c5ffc4" \
width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span \
class="n">linkURL</span> <span class="o">=</span> <span \
class="n">possibleRedirectUrl</span><span class="p">;</span></pre></td> </tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; \
white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">indentation</pre> </div>
<br />
<p>- C.</p>
<br />
<p>On March 28th, 2013, 8:42 p.m. UTC, Aman Madaan wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;"> <tr>
<td>
<div>Review request for Calligra.</div>
<div>By Aman Madaan.</div>
<p style="color: grey;"><i>Updated March 28, 2013, 8:42 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">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 ) </pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">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.) </pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>plugins/textshape/CMakeLists.txt <span style="color: grey">(b2bf9ae)</span></li>
<li>plugins/textshape/ReferencesTool.h <span style="color: \
grey">(21caea8)</span></li>
<li>plugins/textshape/ReferencesTool.cpp <span style="color: \
grey">(46bf06f)</span></li>
<li>plugins/textshape/dialogs/BookmarkLinkInsertionWidget.h <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/BookmarkLinkInsertionWidget.cpp <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/BookmarkLinkInsertionWidget.ui <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/LinkInsertionDialog.h <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/LinkInsertionDialog.cpp <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/LinkInsertionDialog.ui <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SelectBookmark.ui <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SelectBookmarkDialog.h <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SelectBookmarkDialog.cpp <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SimpleBookmarkWidget.h <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SimpleBookmarkWidget.cpp <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SimpleBookmarkWidget.ui <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SimpleCitationBibliographyWidget.cpp <span \
style="color: grey">(2d1c1de)</span></li>
<li>plugins/textshape/dialogs/SimpleCitationBibliographyWidget.ui <span \
style="color: grey">(55699bb)</span></li>
<li>plugins/textshape/dialogs/SimpleLinksWidget.h <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SimpleLinksWidget.cpp <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SimpleLinksWidget.ui <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/SimpleTableOfContentsWidget.h <span style="color: \
grey">(adfd010)</span></li>
<li>plugins/textshape/dialogs/WeblinkInsertionWidget.h <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/WeblinkInsertionWidget.cpp <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/dialogs/WeblinkInsertionWidget.ui <span style="color: \
grey">(PRE-CREATION)</span></li>
<li>words/part/KWView.cpp <span style="color: grey">(8639acd)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/109746/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic