Because when the focus changes, the morph is proposed to redraw itself
Try with a 

    self changed

at the end

Ben

On 27 Dec 2013, at 14:44, Andrei Chis <[email protected]> wrote:

> 
> The change that seems to have caused this problem is in 
> PluggableTextMorph>>selectionInterval: The selectionInterval instance 
> variable of the morph is not set any more; only the model is updated, which 
> doesn't seem to update the morph.
> 
> PluggableTextMorph>>selectionInterval: sel
>       setSelectionSelector
>               ifNil: [ selectionInterval := sel ]
>               ifNotNil: [ self model perform:setSelectionSelector with: sel ]
> 
> This could be solved like bellow, though I'm not sure it is the best 
> solution. 
> If I put a halt in PluggableTextMorph >>inspectIt and hit proceed the the 
> selection is inspected correctly, without this modification.
> 
> PluggableTextMorph>>selectionInterval: sel
>         selectionInterval := sel.
>       setSelectionSelector
>               ifNotNil: [ self model perform:setSelectionSelector with: sel ]
> 
> 
> 
> 
> 
> On Fri, Dec 27, 2013 at 3:11 PM, Marcus Denker <[email protected]> wrote:
> 
> On 27 Dec 2013, at 13:01, Stéphane Ducasse <[email protected]> wrote:
> 
> > Hi guys
> >
> > we should not let the system in such state. We cannot inspect print 
> > expression in the debugger.
> > Does anybody have an idea of the change that broke it?
> >
> Yes:
> 
>         https://pharo.fogbugz.com/f/cases/12144/TextModel-getSelection
> 
> it was added in 655, so for reverting the easiest is to dl 654, do a merge 
> and look at all the changes
> (there are just 4-5 that are not re-catorizations).
> 
> This is on my TODO next…
> 
>         Marcus
> 
> 
> 

Reply via email to