<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/106685/">http://git.reviewboard.kde.org/r/106685/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 2nd, 2012, 6:13 p.m., <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Any reason this is at the Document level and not at the Part level? Feels to me it belongs more into the part than the document.</pre>
</blockquote>
<p>On October 2nd, 2012, 6:21 p.m., <b>Ivan Čukić</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This is really meant to be a document-link (Resource == Document most of the time -> thus ResourceInstance is kinda DocumentInstance) with one small peculiarity - the need for the window id because of share-like-connect.
The second thing is that, for the directory listing, we might need some special handling* so I didn't want to make it work by reporting the location just yet.
* for example, it could report the path the user is browsing, but to report the selected files on request (for share-like-connect)</pre>
</blockquote>
<p>On October 2nd, 2012, 6:23 p.m., <b>Ivan Čukić</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Forget the last two paragraphs, meant this is the Gwenview patch review.</pre>
</blockquote>
<p>On October 2nd, 2012, 8:59 p.m., <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Sure and Part == Document too, i just don't like it in the document, it adds more "ui" dependencies to the core (which already has a few, but the idea is to have less, not more) and besides if i ever do my "okulartotext" command line program that reads a file with okular and exports as text, it won't have a widget (I know there's lots of things to fix, but i'd like to have one less)</pre>
</blockquote>
<p>On October 2nd, 2012, 10:03 p.m., <b>Ivan Čukić</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">What do you think about putting it even above KPart - tied to the Okular application itself?
After some thought, I've decided sharing parts in this case would produce more issues than it would solve.</pre>
</blockquote>
<p>On October 2nd, 2012, 11:28 p.m., <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This way we lose the "activity" thingie when we are embedded in konqueror or some other shell, that from my "i know nothing about activities" sounds right since it would seem it'd be that shell's responsability to be activity enabled, right?</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Yes, it is lost for other shells.
While I'd like it to be in KPart library itself (it is implemented like that in plasma active), the idea has a few issues when the kpart is used in a MDI application (it was ok for Active since we don't have MDIs over there) so this time I decided to go above it.</pre>
<br />
<p>- Ivan</p>
<br />
<p>On October 2nd, 2012, 10:40 p.m., Ivan Čukić wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Okular, Plasma and Albert Astals Cid.</div>
<div>By Ivan Čukić.</div>
<p style="color: grey;"><i>Updated Oct. 2, 2012, 10:40 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;">Okular reports the open/close document events to activity manager daemon.
Side-effect - support for Share-Like-Connect applet.
(and some trailing spaces auto-removal)
</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;">yes</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>shell/shell.cpp <span style="color: grey">(70e2e99)</span></li>
<li>CMakeLists.txt <span style="color: grey">(a057e82)</span></li>
<li>shell/shell.h <span style="color: grey">(393d6c6)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106685/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>