In Safari and Webkit the menu doesn't work quite right.

When the menu is expanded it goes over the top of the page.

When you open and close the menu some parts don't disappear.

CommentFileSizeAuthor
#5 admin-638838.patch1019 bytesmfer
#1 admin-n638838.patch1.04 KBdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review
StatusFileSize
new admin-n638838.patch1.04 KB

I'm seeing the same thing. Basically after it finishes doing the animate() operation to make the width 0px the div.admin-blocks is still left with display:block on the object; adding removeAttr('style') seems to work.

yhahn’s picture

yhahn commented

Can you let me know if this fixes it? I don't have a good Safari environment atm but the other webkit browser I tested in respected body margin changes and not padding changes. I've switched to using margins on the body here:

http://drupal.org/cvs?commit=292592

mfer’s picture

mfer commented
Status: Needs review » Fixed

I just tested this in Safari and Webkit head and it seems to be working. Thanks.

marcoBauli’s picture

marcoBauli commented
Status: Fixed » Needs work

@yhahn: switching paddings to margins seems to fix appearance, still when hiding the left administration menu, it's transparent div stays in place, preventing access to contents on the left side of page.

To reproduce:

- Browse to Modules settings page
- Show Admin menu bar
- Hide it
- Try activating /deactivating modules (checkboxes)

PS: using Rubik admin theme

Thanks

mfer’s picture

mfer commented
Status: Needs work » Needs review
StatusFileSize
new admin-638838.patch1019 bytes

This patch worked for me in Safari and Webkit head. Also works in FF and IE7/8.

yhahn’s picture

yhahn commented

Committed. http://drupal.org/cvs?commit=293516

If this works for you after testing, please close.

mfer’s picture

mfer commented

I'm going to let someone else verify since I wrote it and it works for me :)

briandelicata’s picture

briandelicata commented

Patch worked for me in Safari 4.0.4 (OS X).

mfer’s picture

mfer commented
Status: Needs review » Fixed

Tested on another site and all is well.

Status: Fixed » Closed (fixed)

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