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
>>>
>>>
>>>
>>
>>
>
>
