549988 - [Gecko/20100302 Minefield/3.7a3pre] The Bookmark-Bar-Menu overlays the Bookmark-Folder

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Mehmet]

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.3a3pre) Gecko/20100302 Minefield/3.7a3pre
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.3a3pre) Gecko/20100302 Minefield/3.7a3pre

Hello Firefox-Team,

the Bookmark-Bar-Menu overlays the bookmark-folder (or is to close to it). It could be some pixels down (like in Safari)!

Thanks in advance !

Mehmet

Reproducible: Always

Steps to Reproduce:
1. click on a Bookmarkfolder in the Bookmark-Bar, so that the menu is to see.

Actual Results:  
The Bookmark-Bar-Menu overlays the bookmark-folder (or is to close to it).

Expected Results:  
The Bookmark-Bar-Menu should not overlay the bookmark-folder

Comment 1

[Mehmet]

Attachment: Screenshot of the bug

Comment 2

[Dão Gottwald [:dao]]

Attachment: patch?

This is based on looking at the screenshot, I'm not quite sure if the numbers are right.

Comment 3

[Markus Stange [:mstange]]

Comment on attachment 430458 [details] [diff] [review]
patch?

They're right.

Comment 4

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/14feaa4712a0

Comment 5

[Mehmet]

Attachment: screenshot_white_line_is_missing

Hi together,

thank you for fixing the bug. It looks really good now :-)

But I have noticed, that there is the "white line" missing at the bottom of the Bookmark-Bar-folder when it is pressed.

**Please see enclosed screenshot**

Thank you in advance.

Mehmet

Comment 6

[Markus Stange [:mstange]]

Yeah, the bookmarks bar needs some padding because it's overflow:hidden and cuts the box shadow off. I'll file a bug on that.

Comment 7

[Mehmet]

(In reply to comment #6)
> Yeah, the bookmarks bar needs some padding because it's overflow:hidden and
> cuts the box shadow off. I'll file a bug on that.

Thank you very much :-) Have a nice day !

Comment 8

[Markus Stange [:mstange]]

I've filed bug 551582 on that.