<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/107811/">http://git.reviewboard.kde.org/r/107811/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 9th, 2013, 5:13 p.m. UTC, <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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/107811/diff/5/?file=106093#file106093line52" style="color: black; font-weight: bold; text-decoration: underline;">plasmoid/declarative-plugin/messages-model.cpp</a>
<span style="font-weight: normal;">
(Diff revision 5)
</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; ">public:</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">52</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="kt">bool</span> <span class="n">startOpened</span><span class="p">;</span></pre></td>
</tr>
</tbody>
</table>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Can you talk me through this whole "startOpened" thing?
So there's going to be an option in the UI as to whether all chats start opened or not, but it only has any impact for chats started via the chat plasmoid?</pre>
</blockquote>
<p>On January 10th, 2013, 1:41 a.m. UTC, <b>Aleix Pol Gonzalez</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;">No, that's not for GUI configurability, it's only for telling the front-end code that this new conversation that just appeared has just started needs to be opened instantly. Otherwise we only get a button with the conversation on but not showing.
We say that by passing a specific hint ourselves.</pre>
</blockquote>
<p>On January 12th, 2013, 10:56 p.m. UTC, <b>David Edmundson</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;">So let me confirm the behavior we're trying to get:
- if it's started from this applet it opens instantly
- it it's started from elsewhere (such as the contact list, contact plasmoid, or the other contact started the chat) you get the contact icon, but it doesn't pop out straight away
?</pre>
</blockquote>
<p>On January 12th, 2013, 11:11 p.m. UTC, <b>Aleix Pol Gonzalez</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;">Yes. The thing is that in both cases the conversation channel is opened, the difference is that if you click a pinned is to start saying things right away but when somebody talks to you we need the user to stop doing whatever he's doing before showing the conversation.</pre>
</blockquote>
<p>On January 13th, 2013, 9:01 a.m. UTC, <b>David Edmundson</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;">The reason I asked is because there is a channel->isRequested() which is true if we're the one who initiated the chat.
This would mean if you started from the contact-list-plasmoid it would pop up straight away too.
I think that's better behaviour?
</pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The behaviour is the same, but the code is slightly cleaner, so I decided to use isRequired as you suggested. :)</pre>
<br />
<p>- Aleix</p>
<br />
<p>On January 10th, 2013, 1:41 a.m. UTC, Aleix Pol Gonzalez 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 Telepathy and David Edmundson.</div>
<div>By Aleix Pol Gonzalez.</div>
<p style="color: grey;"><i>Updated Jan. 10, 2013, 1:41 a.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;">I've wanted to use this plasmoid for longtime, I fear that the reason why I'm not using it yet is because I can't start chats from there. The plan is that this will improve the situation, to some extent.
The important part about the patch is that the root element is refactored into a grid that can have different elements. Now I added a button with a contactlist that probably should go, but eventually i'd like to have non-conversation buttons so that one can pin contacts.</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;">Very little, mostly sending this review for starting a discussion on where we'd like to go with this plasmoid.</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>plasmoid/declarative-plugin/CMakeLists.txt <span style="color: grey">(48ba8a7)</span></li>
<li>plasmoid/declarative-plugin/contact-pin.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/declarative-plugin/contact-pin.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/declarative-plugin/conversation-target.h <span style="color: grey">(cd45f2d)</span></li>
<li>plasmoid/declarative-plugin/conversation-target.cpp <span style="color: grey">(f9c285d)</span></li>
<li>plasmoid/declarative-plugin/conversation.h <span style="color: grey">(6eeab86)</span></li>
<li>plasmoid/declarative-plugin/conversation.cpp <span style="color: grey">(152d940)</span></li>
<li>plasmoid/declarative-plugin/conversations-model.h <span style="color: grey">(f9dc047)</span></li>
<li>plasmoid/declarative-plugin/conversations-model.cpp <span style="color: grey">(faaa60b)</span></li>
<li>plasmoid/declarative-plugin/ktp-metatypes.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/declarative-plugin/messages-model.h <span style="color: grey">(233bbfb)</span></li>
<li>plasmoid/declarative-plugin/messages-model.cpp <span style="color: grey">(a33b3c4)</span></li>
<li>plasmoid/declarative-plugin/pinned-contacts-model.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/declarative-plugin/pinned-contacts-model.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/declarative-plugin/qml-plugins.cpp <span style="color: grey">(23a4291)</span></li>
<li>plasmoid/declarative-plugin/telepathy-text-observer.h <span style="color: grey">(241febb)</span></li>
<li>plasmoid/declarative-plugin/telepathy-text-observer.cpp <span style="color: grey">(98dfced)</span></li>
<li>plasmoid/org.kde.ktp-chatplasmoid/contents/config/main.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ChatWidget.qml <span style="color: grey">(ea68f41)</span></li>
<li>plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ContactList.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ConversationDelegate.qml <span style="color: grey">(8a8d851)</span></li>
<li>plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ConversationDelegateButton.qml <span style="color: grey">(PRE-CREATION)</span></li>
<li>plasmoid/org.kde.ktp-chatplasmoid/contents/ui/main.qml <span style="color: grey">(feb766b)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107811/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://git.reviewboard.kde.org/r/107811/s/952/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2013/01/02/ktp-pinned_400x100.png" style="border: 1px black solid;" alt="i am pinned" /></a>
<a href="http://git.reviewboard.kde.org/r/107811/s/953/"><img src="http://git.reviewboard.kde.org/media/uploaded/images/2013/01/02/ktp-pinned1_400x100.png" style="border: 1px black solid;" alt="pin me" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>