Admin toolbar breaks style on Garland when js is disabled.

tic2000 - July 4, 2009 - 15:46
Project:Drupal
Version:7.x-dev
Component:toolbar.module
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Issue tags:Quick fix
Description

If javascript is disabled the toolbar overlaps a part of the header in Garland (on FF3, but probably on others too).

Attached is a screen shot and a patch to fix the problem.

AttachmentSizeStatusTest resultOperations
toolbar-without-js.jpg45.65 KBIgnoredNoneNone
behaving-toolbar-without-js.patch584 bytesIdlePassed: 11562 passes, 0 fails, 0 exceptionsView details | Re-test

#1

keith.smith - July 4, 2009 - 16:10
Title:Admin toolbar brakes style on Garland when js is disabled.» Admin toolbar breaks style on Garland when js is disabled.

#2

xmacinfo - July 4, 2009 - 16:16

#3

alpritt - July 5, 2009 - 21:39
Status:needs review» reviewed & tested by the community

Tested in IE6, IE7, IE8 and Chrome on Windows, and the latest versions of Safari, Firefox and Opera on the Mac, all with JavaScript disabled and enabled.

And the code change makes sense too.

#4

catch - July 5, 2009 - 23:25

I saw a similar overlap with javascript enabled which went away when the page finished loading a few times, looks like a good change.

#5

tic2000 - July 6, 2009 - 06:42

@catch
Yes, because that class is added by toolbar.js when the DOM is ready. The more time it takes to load, more time you have to notice the problem.

#6

Dries - July 7, 2009 - 07:54
Status:reviewed & tested by the community» fixed

Committed to CVS HEAD. Thanks.

#7

System Message - July 21, 2009 - 08:00
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.