<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/124276/">https://git.reviewboard.kde.org/r/124276/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Okular and Albert Astals Cid.</div>
<div>By Nikola Nikolic.</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=342258">342258</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
okular
</div>
<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;">Instead of single bookmarks.xml, there is now bookmarks folder with separated bookmarks for each document and
_bookmarkedFiles.xml that contain information ( folder tags ) about bookmarked documents.
I have also changed BookmarkList class, so that only top level items ( folder titles from _bookmarkedFiles.xml ) are
loaded at begining. Subitems are loaded when needed ( bookmarks searched, tree expanded ).
This makes Okular load faster ( bug 342258 ), but also gives us opportunity to save bookmarks and annotations in the
same folder where document is, addressing problems described in bug 176743 and bug 267350.
There is some change in bookmark metadata that occur during conversion from old bookmarks system to new, example:
before conversion:
<bookmark href="file:///home/nikola/neki_fajl.txt#0;C2:0.499253:0.636771:1">
<title>prvi bookmark</title>
<info>
<metadata owner="http://freedesktop.org">
<bookmark:icon name="text-plain"/>
</metadata>
</info>
</bookmark>
after conversion:
<bookmark href="file:///home/nikola/neki_fajl.txt#0;C2:0.499253:0.636771:1">
<title>prvi bookmark</title>
<info>
<metadata owner="http://freedesktop.org">
<n1:icon name="text-plain" xmlns:n1="http://www.freedesktop.org/standards/desktop-bookmarks"/>
</metadata>
</info>
</bookmark>
I don't know what causes this change, but it seems that Okular doesn't use this info at all.</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;">Everything seems to work, except communication between two Okular processes.
If I open multiple windows in single Okular process, and change bookmarks from one window signal is sent to all
windows from that process. If I have have two different Okular process, and change bookmarks from one window
signal is not sent to window in second process. I don't know how to fix this.</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>core/bookmarkmanager.h <span style="color: grey">(e24efc6)</span></li>
<li>core/bookmarkmanager.cpp <span style="color: grey">(22951b2)</span></li>
<li>core/document.cpp <span style="color: grey">(9d12488)</span></li>
<li>ui/bookmarklist.h <span style="color: grey">(03acd43)</span></li>
<li>ui/bookmarklist.cpp <span style="color: grey">(1db03df)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/124276/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>