<div class="gmail_quote"><div>Hullo,</div><div><br></div><div>I now think that this patch (identical to the &quot;quasi&quot; one sent a few days </div><div>ago) is the fix for my PostIt cursor vs scroll itch. </div><div><br>
</div><div>If I can quickly expand on this here.....</div>
<div><br></div><div>Grepping for all instances of &quot;LayoutPostIts&quot;, turns out that it is called </div><div>by seven places throughout the entire project.</div><div><br></div><div>Applying pseudo-coding to get at a design perspective - to aid my </div>

<div>understanding:</div><div><br></div>sw/source/ui/uiview/viewport.cxx<br><blockquote style="margin:0 0 0 40px;border:none;padding:0px">LayoutPostIts called by OuterResizePixel<br>
</blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><blockquote style="margin:0 0 0 40px;border:none;padding:0px">= when window changes, if postits visible, if cursor on a <br>
postit, that postit must remain in view</blockquote></blockquote><br>sw/source/core/view/viewsh.cxx<blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>by VisPortChanged</div>
</blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>= when visual port changes, and if postits visible, </div>
<div>if cursor on a posit, that postit must remain in view</div></blockquote></blockquote><div><br>sw/source/ui/docvw/PostItMgr.cxx</div><blockquote style="margin:0 0 0 40px;border:none;padding:0px">
<div>Delete</div></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><blockquote style="margin:0 0 0 40px;border:none;padding:0px">
<div>= [n/a]</div></blockquote></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>Delete [author notes overloaded version]</div></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px">

<blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>= [n/a]</div></blockquote></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px">
<div>Hide</div></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><blockquote style="margin:0 0 0 40px;border:none;padding:0px">
<div>= [n/a]</div></blockquote></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>Show</div></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px">
<blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>= when postits first appear, i.e. on document load, cursor </div><div>is on document proper anyway.  otherwise, if postits </div>
<div>visible, if cursor on a postit, that postit must remain in view</div></blockquote></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>CalcHdl [event handler]</div>
</blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>= for whatever event we are interested in, if postits visible,</div>
<div>if cursor on a postit, that postit must remain in view.</div></blockquote><br><br></blockquote>Unless I&#39;ve missed something more fundamental, IMHO the pseudo<div>code above makes little sense.<br><div><blockquote style="margin:0 0 0 40px;border:none;padding:0px">

<br></blockquote>Removing the code by this patch gives us scrolling how it is for the <div>document proper.  (The postit containing the cursor is brought back to</div></div></div><div>view by the user typing on the keyboard/arrows as per elsewhere)</div>

<div><br></div><div>However I&#39;m more than OK if this patch needs to be &#39;shelved&#39; for now  :)</div><div><br></div><div>Cheers</div><div>Andy</div></div>