109280 - [FIXr]onFocus and onBlur do work within fieldset only after resizing the window

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect, P2)

Description

[Diethelm Schneider]

From Bugzilla Helper:
User-Agent: Mozilla/4.0 (compatible; MSIE 5.01; Windows NT 5.0)
BuildID:    2001110209

within fieldsets there is no reaction visible using onFocus and onBlur, only if 
you resize the window there is effort.
without fieldset onFocus and onBlur do function without problems.

Reproducible: Always
Steps to Reproduce:
1.Try http://selfhtml.teamone.de/javascript/sprache/anzeige/onfocus.htm
2.Try http://selfhtml.teamone.de/javascript/sprache/anzeige/onblur.htm
3.Try 
http://selfhtml.teamone.de/javascript/objekte/anzeige/htmlelemente_legend.htm
4. Try the last one resizing the window

Comment 1

[Christopher Hoess (gone)]

Confirmed. -> Form Controls, but maybe DOM HTML.

Comment 2

[rods (gone)]

strange

Comment 3

[John Keiser (jkeiser)]

-> default owner.  This is still a bug.  May be focus; CC'ing saari in case it is.

Comment 4

[Boris Zbarsky [:bzbarsky]]

This has nothing to do with focus and everything to do with the fact that the
legend frame's align attribute change triggers no reflow.  

Comment 5

[Boris Zbarsky [:bzbarsky]]

Attachment: Partial fix

There still seems to be a painting issue here, but at least the reflow
happens...

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Where's the code that maps align to layout without going through style?

Comment 7

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

...but it might be nice to map align to style at some point instead of having
nsLegendFrame::GetAlign.

Comment 8

[Boris Zbarsky [:bzbarsky]]

Doh.  I forgot to cc myself.  We'd basically need another internal style
property a la the table ones to map this into style, right?  Mapping align on a
legend into float would be pretty wrong, I think.

Comment 9

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

What does it do?  Maybe it maps to auto margins?

Comment 10

[Boris Zbarsky [:bzbarsky]]

Hmm... yeah, that may work.  I'd really like to move away from the
special-purpose frames for legend and fieldset, btw... I just haven't figured
out a good way to map our current behavior into CSS so far (in particular, the
fact that the legend's width can affect the fieldset width).

Comment 11

[Boris Zbarsky [:bzbarsky]]

Bug 230154 filed on the style mapping issue.

Bug 230155 filed for the painting problem.

Patch checked in for 1.7a.

Comment 12

[Bernd]

just a short warning chris k. was here, aka the relationship between legend and
fieldsets are very similiar to captions and tables and the outer table frame. 

Comment 13

[Boris Zbarsky [:bzbarsky]]

You mean changing align on a legend should really reflow the fieldset
completely?  As written, that may indeed be the case.. :(