[Pharo-project] misleading behaviour of TextModel>>setSelection: in Spec
Hi, I am playing with spec hit some issues with TextModel>>setSelection: First, unlike any other method in TextModel, setSelection: only has an effect if you call it after sending openWithSpec to the model. This is because it sets the selection directly on the morph widget. Second if you set the selection and widget hasn't got the focus it won't have any visual effect. This is because it uses PluggableTextMorph>>selectionInterval: which just updates the interval without doing any visual update. So shall we classify these as bugs? Cheers, Andrei
I think it's a bug. At least it's not the expected behaviour :) Ben On Jan 31, 2013, at 7:40 PM, Andrei Vasile Chis <chisvasileandrei@gmail.com> wrote:
Hi,
I am playing with spec hit some issues with TextModel>>setSelection:
First, unlike any other method in TextModel, setSelection: only has an effect if you call it after sending openWithSpec to the model. This is because it sets the selection directly on the morph widget.
Second if you set the selection and widget hasn't got the focus it won't have any visual effect. This is because it uses PluggableTextMorph>>selectionInterval: which just updates the interval without doing any visual update.
So shall we classify these as bugs?
Cheers, Andrei
Ok. I'll open an issue then. Andrei On Thu, Jan 31, 2013 at 8:41 PM, Benjamin < benjamin.vanryseghem.pharo@gmail.com> wrote:
I think it's a bug. At least it's not the expected behaviour :)
Ben
On Jan 31, 2013, at 7:40 PM, Andrei Vasile Chis < chisvasileandrei@gmail.com> wrote:
Hi,
I am playing with spec hit some issues with TextModel>>setSelection:
First, unlike any other method in TextModel, setSelection: only has an effect if you call it after sending openWithSpec to the model. This is because it sets the selection directly on the morph widget.
Second if you set the selection and widget hasn't got the focus it won't have any visual effect. This is because it uses PluggableTextMorph>>selectionInterval: which just updates the interval without doing any visual update.
So shall we classify these as bugs?
Cheers, Andrei
I created two :) http://code.google.com/p/pharo/issues/detail?id=7382 http://code.google.com/p/pharo/issues/detail?id=7383 For the second I also uploaded a possible fix. Andrei On Thu, Jan 31, 2013 at 8:44 PM, Andrei Vasile Chis < chisvasileandrei@gmail.com> wrote:
Ok. I'll open an issue then.
Andrei
On Thu, Jan 31, 2013 at 8:41 PM, Benjamin < benjamin.vanryseghem.pharo@gmail.com> wrote:
I think it's a bug. At least it's not the expected behaviour :)
Ben
On Jan 31, 2013, at 7:40 PM, Andrei Vasile Chis < chisvasileandrei@gmail.com> wrote:
Hi,
I am playing with spec hit some issues with TextModel>>setSelection:
First, unlike any other method in TextModel, setSelection: only has an effect if you call it after sending openWithSpec to the model. This is because it sets the selection directly on the morph widget.
Second if you set the selection and widget hasn't got the focus it won't have any visual effect. This is because it uses PluggableTextMorph>>selectionInterval: which just updates the interval without doing any visual update.
So shall we classify these as bugs?
Cheers, Andrei
participants (2)
-
Andrei Vasile Chis -
Benjamin