<table><tr><td style="">poboiko created this revision.<br />poboiko added a reviewer: dvratil.<br />poboiko requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D29289">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The code is mostly shared with <tt style="background: #ebebeb; font-size: 13px;">KJotsEdit</tt>; however, <tt style="background: #ebebeb; font-size: 13px;">KJotsBrowser</tt> should also support for<br />
anchor links like <tt style="background: #ebebeb; font-size: 13px;">#page_XXX</tt>, which are generated by the html template.</p>

<p>The link message in statusbar is no longer needed, as user already have relevant feedback<br />
in a form of tooltip (which is more obvious than expecting link to appear in statusbar).<br />
It also didn't recognize <tt style="background: #ebebeb; font-size: 13px;">akonadi:</tt> links, and didn't work in browser mode.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Code compiles, tooltips appear in browser mode too</p></div></div><br /><div><strong>BRANCH</strong><div><div>no-statusbar (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D29289">https://phabricator.kde.org/D29289</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/KJotsMain.cpp<br />
src/KJotsMain.h<br />
src/kjotsbrowser.cpp<br />
src/kjotsbrowser.h<br />
src/kjotspart.cpp<br />
src/kjotspart.h<br />
src/kjotswidget.cpp<br />
src/kjotswidget.h</div></div></div><br /><div><strong>To: </strong>poboiko, dvratil<br /><strong>Cc: </strong>kde-pim<br /></div>