Closed Bug 189292 Opened 22 years ago Closed 22 years ago

Rule [container:hover child {...}] doesn't work without a border applied to [container:hover]

Categories

(Core :: CSS Parsing and Computation, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Maniac, Assigned: dbaron)

References

()

Details

Attachments

(3 files)

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.3b) Gecko/20030114
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.3b) Gecko/20030114

I have a stylesheet with rules for "display"ing child elements of some container
when this container is :hovered (realization of CSS menus). It used to work in
previous versions of mozilla (sorry, can't specify more precisely). In recent
nightly it refuses to work until I specify some "border" on a container.

P.S. Sorry I cannot produce a testcase because I can't reproduce the broken
behaviour on any synthetic example :-(

Reproducible: Always

Steps to Reproduce:
1. Look at the URL
2. Try hovering over the menu at the top
3. Look at the stylesheet http://softwaremaniacs.org/New/Version1/Modern.CSS
4. Read a note to bugzilla emphasized in the file
5. Comment bordering rule

Actual Results:  
CSS menus stop working

Expected Results:  
CSS menus should work as at good old times :-)

I suspect this is a regression from a recently fixed bug about applying :hover
only to 'A' elements in quirks mode (can't supply # - search is broken now). But
my page is in strict HTML and besides I'm confused with what borders have to do
with this. Anyway it's just a guess...
You mean bug 169078?  There's really not much I can do without a testcase.  I
doubt it, considering that should only affect .class:hover and it should only
affect quirks mode.
Oh, there's a URL.  For some reason I thought there wasn't.
The most I could reproduce when trying to create a testcase is a non-working
:hover rule:

<Style Type="text/css">
  Body {Color:Black; Background-Color:White;}
  Div:Hover P {Color:Red;}
</Style>
<Body>
<Div>
  <P>This should be red on hovering.
</Div>

But I couldn't reproduce that _bordering_ a container helps. So I decided not to
include a testcase in order to avoid a confusion... 
It's a regression from bug 188734 (the nsCSSFrameConstructor patch).
Attached patch patchSplinter Review
This was a simple errror -- a misplaced closing brace.
Fix checked in to trunk, 2003-01-16 05:59 PDT.

Thanks for noticing the bug.
Status: UNCONFIRMED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Great! Thank you for fixing it so fast.
According to bug 191974, comment 18 this caused a crash regression.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: