Hi,

After reading Andreas mail I went to make a changeset with Andreas'
suggestion but while looking at the code it hit me that Ken's fix were
already doing as Andreas had said. That is. Before firing a menuItem
restoring the keyboard focus.

So I went back to my example of why it wasn't working as expected and I
think it was because another related bug that I discovered yesterday. The
one that I posted on: '[FIX][BUG]New Paragraph wasn't initialized
correctly'.

So. The changeset that I am sending now is just Ken's changeset (v2) plus
the same fix but for the FillInTheBlankMorph and my fix that I described in
'[FIX][BUG]New Paragraph wasn't initialized correctly' because these 3 are
needed for the same BUG and I think this is still a reasonably small
changeset.

I manually tested it trying the different things we were talking on this
thread. The problem with the WorldMenu is still there though.

Regards,
Hernán

> -----Original Message-----
> From: squeak-dev-bounces at lists.squeakfoundation.org
> [mailto:squeak-dev-bounces at lists.squeakfoundation.org]On Behalf Of
> Andreas Raab
> Sent: Monday, April 12, 2004 5:40 PM
> To: The general-purpose Squeak developers list
> Subject: Re: [FIX][BUG] Hidden text cursor ( v4 - a more general
> solution? )
>
>
> > You are right about what you said but we were doing something like this:
> >
> > 1. we remembered the current focus holder.
> > 2. then we activated the MenuMorph, FIllInTheBlank and thus setting
> > a new focus holder.
> > 3. and finally we restored the focus to the morph on 1.
> >
> > But this wasn't right. I will give you this escenario as an example.
>
> Actually, the above sequence is perfectly right. You just have to
> make sure
> that the focus transitions happen at the "right point", e.g.,
>
> > The focus is on a browser code pane and you right-click on a
> category list
> > morph. The menu that popups obtain focus and you select 'add category'
> then
> > new. A FillInTheBlank popups and get focus. The previous MenuMorph is
> > deleted. You enter the new category name and accept it. The
> FillInTheBlank
> > ends and return the focus to the .... MenuMorph which is no more on the
> > world.
>
> This would be the wrong sequence. The right one goes like this:
> * the focus is on a browser code pane and you right-click on a
> category list
> * the menu pops up aquires focus and remembers the previous one
> * you select "add category"
> [--> here is the difference <--]
> * the menu passes the focus back to list
> * the menu *THEN* fires, closes and the FITBM opens
> [--> end of difference <--]
> * the FITBM aquires focus
> * you enter the new name and accept
> * the FITBM passes focus back to the category list
> * the FITBM *THEN* closes and continues
> * etc.
>
> Cheers,
>   - Andreas
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: HiddenCursorFix-hpt.1.cs
Type: application/octet-stream
Size: 5773 bytes
Desc: not available
Url : http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20040413/c118c3b2/HiddenCursorFix-hpt.1.obj