<table><tr><td style="">davidhurka updated this revision to Diff 59149.<br />davidhurka marked an inline comment as done.<br />davidhurka edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-5kyqhtgtkwccec5/">(Show Details)</a><br />davidhurka added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D21266">View Revision</a></tr></table><br /><div><div><p>Removed references to pages from methods.</p>

<p>Now they refer to a reference area, and the description of the normalized coordinate system now explains how mapping to a “reference area” works, and how parameters xScale and yScale are related.</p>

<p>Also added another example section, now covers transformation both to and from a page coordinate system. The second example touches rotation, I can remove that if demanded.</p>

<p>ObjectRect still uses pages as reference area, because I consider ObjectRect coordinates  page related. (Document objects are located on pages.) For consistency, I can remove these references there too, if demanded.</p>

<p>Annotation::Quad is related, but is already pretty well documented. Just clarified some things I consider relevant. (Only 4 points instead of 8 points, and things which are directly related to Bug 334297.)</p>

<p>I think it makes sense to move text reordering to another patch.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>coordinates are used.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Also found some potential bugs in functions which are currently unused.<br />
<br />
Todo:<br />
There are these ObjectRect classes, and I don’t know what makes them</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">TODO:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">better than NormalizedRect in</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">ObjectRect subclasses call</span> the<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">ir current applications. <br />
<br />
Also see my (inline) comments</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">mselves rectangles, although they aren’t neccessarily</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D21266?vs=58380&id=59149">https://phabricator.kde.org/D21266?vs=58380&id=59149</a></div></div><br /><div><strong>BRANCH</strong><div><div>improve-area-classes-documentation</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21266">https://phabricator.kde.org/D21266</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>core/annotations.h<br />
core/area.h</div></div></div><br /><div><strong>To: </strong>davidhurka, Okular<br /><strong>Cc: </strong>aacid, okular-devel, joaonetto, tfella, ngraham, darcyshen<br /></div>