**thanks** guys for looking into that because this is blocking. Stef
> I think that when the model is notified, it will set it in the morph > So it will happen twice > > Ben > > On 27 Dec 2013, at 15:52, Andrei Chis <[email protected]> wrote: > >> Wouldn't then explicitly setting the selection of the morph before setting >> it in the model be ok? >> >> PluggableTextMorph>>selectionInterval: sel >> selectionInterval := sel. >> setSelectionSelector >> ifNotNil: [ self model perform:setSelectionSelector with: sel ] >> >> >> On Fri, Dec 27, 2013 at 4:28 PM, Benjamin >> <[email protected]> wrote: >> ok :( >> Then I run out of brilliant idea :P >> >> >> Ben >> >> On 27 Dec 2013, at 15:25, Andrei Chis <[email protected]> wrote: >> >>> Adding 'self changed' at the end of the current >>> PluggableTextMorph>>selectionInterval: doesn't solve the problem. >>> >>> >>> On Fri, Dec 27, 2013 at 4:15 PM, Benjamin >>> <[email protected]> wrote: >>> 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 >>>> >>>> >>>> >>> >>> >> >> >
