Bug 15285

Summary:

-webkit-box-shadow not showing on element that was hidden, is displayed

Product:

WebKit

Reporter:

Stephen Rose <pixelpp>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

mitz

Priority:

Version:

523.x (Safari 3)

Hardware:

OS:

All

Attachments:

Description	Flags
Test case	none
A screen grab of the issue	none

Stephen Rose

Reported 2007-09-25 23:35:07 PDT

I was doing a test of the new support for -webkit-box-shadow, with a UL powered drop down menu - where the child menu's (UL) of the main UL are display:none; , and on li:hover they are display:block;. I put a box-shadow on the main UL, and child UL's. and without the display:hidden, the shadow works, but with it, it does not.

Attachments
Test case (1.11 KB, text/html) 2007-09-26 01:02 PDT, mitz	no flags	Details
A screen grab of the issue (71.32 KB, image/jpeg) 2007-09-26 18:16 PDT, Stephen Rose	no flags	Details
View All Add attachment proposed patch, testcase, etc.

mitz

Comment 1 2007-09-26 00:27:52 PDT

Stephen, thank you for the bug report. Can you attach or link to a test case demonstrating the bug?

Stephen Rose

Comment 2 2007-09-26 00:58:43 PDT

Okay cool.. basicly.. my (x)HTML/CSS is as follows: p.s. thanks for all of the awesome work you do... I wish safari had a 90% marketshare... it would make my job.... well.. fun. instead of painfull with IE hacks! <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd"> <html xmlns="http://www.w3.org/1999/xhtml"> <head> <title>Menu/Tab Test</title> <style type="text/css"> body{ margin:0; padding:0; font:14px arial; } .menu { height:25px; } .menu, .menu ul{ margin:0; padding:0; list-style:none; background:#444; border:1px solid #000; -webkit-box-shadow: 0px 5px 5px #777; } .menu li{ float:left; margin-right:5px; } .menu li a{ display:block; line-height:15px; padding:5px; color:#fff; } .menu li ul{ display:none; height:auto; } .menu li:hover ul{ display:block; -webkit-box-shadow: 0px 5px 5px #777; } .menu li ul li{ float:none; } .menu li ul{ position:absolute; } </style> </head> <body> <ul class="menu"> <li><a href="#page1">Page 1</a></li> <li><a href="#page2">Page 2</a></li> <li><a href="#page3">Page 3</a> <ul> <li><a href="#subpage1">Sub Page 1</a></li> <li><a href="#subpage2">Sub Page 1</a></li> </ul> </li> <li><a href="#page4">Page 4</a></li> </ul> </body> </html>

mitz

Comment 3 2007-09-26 01:02:14 PDT

Created attachment 16396 [details] Test case Test case from comment #2 as an attachment.

mitz

Comment 4 2007-09-26 01:08:18 PDT

I can't reproduce the bug on my configuration. Are you using the latest nightly build from <http://nightly.webkit.org>?

Stephen Rose

Comment 5 2007-09-26 16:36:33 PDT

It looks like it detected my hardware, (PC) but not the OS, I am running Safari for Windows on a PC... I am pretty sure that is why it is happening ... it is a PC running Windows. Sorry about the confusion, I guess you guys are doing different things on windows...as the shadow support is probably built in on the mac right? Thanks

Stephen Rose

Comment 6 2007-09-26 16:39:17 PDT

More info on the bug: When you hover over the test case's menu "Page 3" - the one with the sub menu...and switch to a new tab - ie using the shortcut ctrl+shift+ ], and back... it appears - and then does not disapear when hovered off.

Matt Lilek

Comment 7 2007-09-26 16:47:25 PDT

(In reply to comment #6) > More info on the bug: > > When you hover over the test case's menu "Page 3" - the one with the sub > menu...and switch to a new tab - ie using the shortcut ctrl+shift+ ], and > back... it appears - and then does not disapear when hovered off. > I see that with Safari 3.0.3 Beta but my local build of r25755 works fine. Have you tried this with a nightly from <http://nightly.webkit.org/>?

Stephen Rose

Comment 8 2007-09-26 18:08:05 PDT

I thought that I was using the nightly build, how do i find out? I see 3.0.3 (522.15.5) I downloaded the nightly ZIP, ran the .bat (with safari closed) and it launched safari, i even tryed replaceing the WebKit.resources and WebKit.dll files in the safari program files dir. is this not how it works?? Anyway.. with the above steps taken (supposedly using the nightly builds)... I still see the bug.

Stephen Rose

Comment 9 2007-09-26 18:16:18 PDT

Created attachment 16410 [details] A screen grab of the issue

Stephen Rose

Comment 10 2007-09-28 05:58:16 PDT

I was having troubles using the nightly build, I don't believe that I was using the lastest version, sorry. The problem is not shown in the lastest build, I have marked this as fixed. Thanks. How awesome will it be to do a menu system like the above in the future for all browsers... shadows and all... amazing. thank you so much.

Note You need to log in before you can comment on or make changes to this bug.