363056 - [Pinstripe] Restrict popupmenus to a certain height

Status: NEW

(Core :: Widget: Cocoa, defect)

Description

[Håkan Waara]

In Aqua, you can't make popupmenus taller than the default height. If you try to in Interface Builder, then the clicking region will be taller, but the actual popup image won't (because it looks bad).

I'll post a screenshot to demonstrate.

Comment 1

[Håkan Waara]

Attachment: Popupmenus should never be this tall

This is a screenshot from some XUL. Popupmenus shouldn't be able to be this tall.

Comment 2

[Håkan Waara]

Attachment: A normal-sized popupmenu with the right height

Comment 3

[Stefan [:stefanh]]

Iirc the height is hardcoded. It would probably be best to not set any height at all and let the font-size determine the height instead.

Comment 4

[Håkan Waara]

(In reply to comment #3)
> Iirc the height is hardcoded. It would probably be best to not set any height
> at all and let the font-size determine the height instead.

As long as it looks good, yeah... BTW, why is the default font in XUL documents the one in the screenshot, and not the system font?

Comment 5

[Stefan [:stefanh]]

(In reply to comment #4)
> (In reply to comment #3)
> > Iirc the height is hardcoded. It would probably be best to not set any height
> > at all and let the font-size determine the height instead.
> 
> As long as it looks good, yeah... BTW, why is the default font in XUL documents
> the one in the screenshot, and not the system font?
> 

The font that you see is the font you have specified in your browser settings. This is probably correct. This is also why it sometimes can be dangerous to hard-code sizes - imagine that a xul developer creates a menulist and the document that the menulist is in has a 7px font. The "menupopup" is then 20px high with a 7px font in and when you open the menulist, font-size in menuitems is Apple's menu font :-/