<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/101993/">http://git.reviewboard.kde.org/r/101993/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On July 18th, 2011, 3:19 p.m., <b>Sven Langkamp</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;">Having two windows side by side introduces lots of other problem:
-It would bring back the jumping when switching views, because dockers would be hidden and shown all the time.
-You can't have the dockers on one side of the screen as there are two different windows.
-You need a good window manager, so that on splitting the windows appear side by side.
Some of the problems with the current system are there because we tried to avoid those problems e.g. we switch the dockers internally to avoid the jumping. The problem you pointed out a purely implementation problems and not problems with the design of the UI. I don't think that we should change the UI just because the implementation isn't working.</pre>
</blockquote>
<p>On July 19th, 2011, 11:36 a.m., <b>Boudewijn Rempt</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, I agree in principle... But the design limits the usefulness because we cannot have two different documents in one window in any case, and given the problems we have had over the years with this feature I doubt that we will manage to have it fixed. And there are so many crashes and every new docker makes it likely we'll get one more crash that I don't think trying to fix this ancient code is worth it. It's not like other apps all have this feature -- LibreOffice doesn't. But I'd welcome a patch that really fixes split view -- though whether that should be based on the current, very ancient, code, I don't know.</pre>
</blockquote>
<p>On July 21st, 2011, 8:07 a.m., <b>Boudewijn Rempt</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;">Interestingly, MyPaint is adding split view functionality... But they do it sort of right: they can have both vertical and horizontal splits in one window. I'm wondering whether it wouldn't be a good idea to make a fresh start for Calligra, with something that allows all those missing features.</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;">As far as I can see MyPaint does a few things differently. Views only control how the image is display, everything else it shared between the views. In Calligra there is a hard seperation and only the document itself is shared between the view. I wonder if it wouldn't be better to share at least some (or maybe everything) things between the views.
The main problem is the old KPart approach to do the splitting. That really inflexible and doesn't allow us to fix some of the problem that we are currently facing.</pre>
<br />
<p>- Sven</p>
<br />
<p>On July 18th, 2011, 1:03 p.m., Boudewijn Rempt 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 Calligra.</div>
<div>By Boudewijn Rempt.</div>
<p style="color: grey;"><i>Updated July 18, 2011, 1:03 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 removes the buggy split-view functionality from KoMainWindow. split-view has a number of problems:
* crashes when removing views
* duplicates the actions so when a view is split you cannot e.g. delete a shape with the delete key
* it is limited in that it can only show views on one document, instead of splitting the view and then making it possible to show several documents side-by-side
After discussion on irc, we felt it was better to let the window manager handle side-by-side views. To that end, a patch to optionally hide dockers if the window loses focus will follow.</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;">Started several apps, opened and closed documents.</pre>
</td>
</tr>
</table>
<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=239309">239309</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=265200">265200</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=277953">277953</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=278000">278000</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>flow/part/flow.rc <span style="color: grey">(bd2913a)</span></li>
<li>karbon/data/karbon.rc <span style="color: grey">(18c7156)</span></li>
<li>kpresenter/part/stage.rc <span style="color: grey">(d48fd3c)</span></li>
<li>kpresenter/stage.rc <span style="color: grey">(3e0f53d)</span></li>
<li>krita/krita.rc <span style="color: grey">(3d5d85d)</span></li>
<li>libs/main/KoDocument.cpp <span style="color: grey">(515911a)</span></li>
<li>libs/main/KoMainWindow.h <span style="color: grey">(d44aff6)</span></li>
<li>libs/main/KoMainWindow.cpp <span style="color: grey">(c629827)</span></li>
<li>tables/tables.rc <span style="color: grey">(0fde1b6)</span></li>
<li>words/part/words.rc <span style="color: grey">(4e10d36)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101993/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>