<div dir="ltr">Hi Karl,<div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 23, 2014 at 10:15 AM, karl ramberg <span dir="ltr"><<a href="mailto:karlramberg@gmail.com" target="_blank">karlramberg@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Before this change, moving the mouse out of a text or list pane meant it lost keyboard focus.<div>Even if the <span style="color:rgb(80,0,80)">Preferences mouseOverForKeyboardFocus was false.</span></div>
<div>
<span style="color:rgb(80,0,80)">With this change you have to click the pane you want tho have keyboard focus.</span></div><div class="gmail_extra"><br></div><div class="gmail_extra">I find working with this change is less aggravating since I very often bump the mouse out of small text boxes and panes. </div>
</div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">
<div class="gmail_extra"><br></div><div class="gmail_extra">It also means the preference does what it say it will do.</div></div></blockquote><div><br></div><div>This sounds like manna from heaven. I vote for including this in trunk.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">Karl<br></div><div class="gmail_extra"><br><div class="gmail_quote">
On Mon, Jun 23, 2014 at 5:18 PM, Chris Muller <span dir="ltr"><<a href="mailto:asqueaker@gmail.com" target="_blank">asqueaker@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Can you explain what effect this has for regular interaction with an<br>
image? It's not obvious to me.<br>
<div><div><br>
On Sun, Jun 22, 2014 at 10:04 AM, <<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>> wrote:<br>
> A new version of Morphic was added to project The Inbox:<br>
> <a href="http://source.squeak.org/inbox/Morphic-kfr.722.mcz" target="_blank">http://source.squeak.org/inbox/Morphic-kfr.722.mcz</a><br>
><br>
> ==================== Summary ====================<br>
><br>
> Name: Morphic-kfr.722<br>
> Author: kfr<br>
> Time: 22 June 2014, 4:59:39.758 pm<br>
> UUID: fffb2b9e-3b71-f341-9042-4e35e7cc19fa<br>
> Ancestors: Morphic-cmm.721<br>
><br>
> Make keyboard focus respect Preferences mouseOverForKeyboardFocus. With this change one has to click for keyboard fokus.<br>
><br>
> =============== Diff against Morphic-cmm.721 ===============<br>
><br>
> Item was changed:<br>
> ----- Method: PluggableListMorph>>mouseUp: (in category 'events') -----<br>
> mouseUp: event<br>
> "The mouse came up within the list; take appropriate action"<br>
> | row |<br>
> row := self rowAtLocation: event position.<br>
> "aMorph ifNotNil: [aMorph highlightForMouseDown: false]."<br>
> model okToChange<br>
> ifFalse: [^ self].<br>
> "No change if model is locked"<br>
> row = self selectionIndex<br>
> ifTrue: [(autoDeselect ifNil: [true]) ifTrue:[row = 0 ifFalse: [self changeModelSelection: 0] ]]<br>
> ifFalse: [self changeModelSelection: (self modelIndexFor: row)].<br>
> + event hand newKeyboardFocus: self.<br>
> + hasFocus := true.<br>
> Cursor normal show!<br>
><br>
> Item was changed:<br>
> ----- Method: ScrollPane>>handlesMouseOver: (in category 'event handling') -----<br>
> handlesMouseOver: evt<br>
> "Could just ^ true, but this ensures that scroll bars won't flop out<br>
> if you mouse-over appendages such as connecting pins."<br>
> self flag: #arNote. "I have no idea how the code below could've ever worked. If the receiver does not handle mouse over events then it should not receive any #mouseLeave if the mouse leaves the receiver for real. This is because 'evt cursorPoint' describes the *end* point of the movement and considering that the code would return false if the move ends outside the receiver the scroll bars should never pop back in again. Which is exactly what happens with the new event logic if you don't just ^true. I'm leaving the code in for reference - perhaps somebody can make sense from it; I sure cannot."<br>
> + ^Preferences mouseOverForKeyboardFocus<br>
> - ^true<br>
> "<br>
> | cp |<br>
> cp := evt cursorPoint.<br>
> (bounds containsPoint: cp)<br>
> ifTrue: [^ true]<br>
> ifFalse: [self submorphsDo:<br>
> [:m | (m containsPoint: cp) ifTrue:<br>
> [m == scrollBar<br>
> ifTrue: [^ true]<br>
> ifFalse: [^ false]]].<br>
> ^ false]<br>
> "!<br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div></div>
<br><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>best,<div>Eliot</div>
</div></div>