<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/107617/">http://git.reviewboard.kde.org/r/107617/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ok pushed publish a bit too soon there..
What I meant to say is that you can have all sorts of overlappping ranges, and in tables such ranges are even weirder (what is numerically within is not within at all). The concept is thus broken. It's not possible to really find a true solution.
The current code is wrong too ofcourse</pre>
<br />
<p>- C.</p>
<br />
<p>On December 6th, 2012, 8:37 p.m., Friedrich W. H. Kossebau 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 and C. Boemann.</div>
<div>By Friedrich W. H. Kossebau.</div>
<p style="color: grey;"><i>Updated Dec. 6, 2012, 8:37 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;">Currently the code does not really seem to do what the API dox proposes: "find the xmlid of the semitem that is at or surrounding the cursor given. As with findExtent() this will be only the most nested semitem."
Attached patch changes that, in a naive approach. Correct one?
Or does the API dox need adaption?
Okay to commit to 2.6 and forward port to master?</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;">Selecting tables in orpheus works for me with this patch.
But there is a problem: on exchanging the cell contents the bookmark start moves behind the new content in the first cell. So putting the cursor onto the new content will fall outside of the bookmark range. Similar problem at the end. This needs to be fixed separately.</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>libs/main/rdf/KoDocumentRdf.cpp <span style="color: grey">(49275c4)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107617/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>