9305 - When text input control loses focus, insertion caret remains

Status: VERIFIED FIXED

(Core :: Layout, defect, P3)

Description

[cpratt]

Build ID: 1999070608
Platform: Windows NT

To reproduce:
- Go to the above URL (or use the attached test case)
- Click into the text input control
- Note the blinking insertion cursor
- Click on the button

Result: The text input control still has a blinking insertion cursor in it, AND
the button now has focus (visually as well).

Expected result: Because the text input control has lost focus, the insertion
cursor should go away.

As the BUTTON element is not supported in Communciator 4.x, you can use MSIE 5
as a reference for the correct behavior.

Comment 1

[cpratt]

Attachment: Test case (same as http://schist/forms/button1.html)

Comment 2

[rickg]

Kevin -- a widget bug you may care about.

Comment 3

[Kevin McCluskey (gone)]

Steve, re-assigning to you as something to check for when ender-based text
fields land.

Comment 4

[buster]

it's a dup, just need to look up the number.  problem is we're not getting focus
events through webshell interfaces yet.

Comment 5

[Simon Fraser [no longer active]]

*** Bug 4502 has been marked as a duplicate of this bug. ***

Comment 6

[buster]

fixed in my tree, waiting for M10 checkins to open.

Comment 7

[Kathleen :Brade]

fixing summary

Comment 8

[buster]

the editor is never getting a blur event when a GFX-rendered button is clicked
on.

Comment 9

[buster]

assigned to joki.  tom says there's a general problem that in some cases, events
on GFX rendered widgets get eaten before they become focus/blur events.  Once
tom fixes this general problem, I can revisit this specific problem if it
persists.

Comment 10

[Eric Pollmann]

I'm not seeing this bug any more.
Steve, you fixed this, right?

Comment 11

[buster]

Hyatt and Saari actually fixed this.  They changed buttons so that they no
longer steal focus from the text control.

Comment 12

[rhys]

This also happens when another text field is selected (in M11).

Comment 13

[leger]

M11 out the door...moving to M12.

Comment 14

[joki (gone)]

I'm betting this is an issue with focus/blur notifications getting screwed
up for text widgets.  I'm gonna sling this by saari as I happen to know he's
working in just this area right now.  And didn't I hear someone call him
focus-boy the other day?

Comment 15

[Peter Trudelle]

Mass-moving non-PDT+ bugs to M13

Comment 16

[saari (gone)]

Fixed

Comment 17

[Chris Petersen]

With the Dec 15th Linux build, this problem has been fixed.