123 - Form menus not wide enough

Status: VERIFIED WONTFIX

(MozillaClassic Graveyard :: XFE, defect, P2)

Description

[Darrell Kindred]

Created by Darrell Kindred (dkindred=[e-mail]) on Wednesday, April 8, 1998 5:18:59 PM PDT
Additional Details :
Using Motif 1.2.4 under Linux, menus in HTML forms are not
wide enough for both the text of the longest item and the
little arrow button on the right.  It looks like the width
of the arrow is not taken into account.  For an example with
screenshot, see
  http://www.cs.cmu.edu/People/dkindred/bugs/menu-width.html
Updated by Ramiro Estrugo ([e-mail]) on Wednesday, April 29, 1998 2:46:32 AM PDT
Additional Details :
Assign to ramiro.

Yes the combo box sucks and has lots of bugs.  Thanks.

Comment 1

[Felix]

Hi Ramiro,

I believe I've tracked this bug down, and I might have a fix or at least an
analysis as soon as I can find some more time.

In fact, I was just wondering about the comment that you made on replacing
ComboBox at all. If you have something else to plug in already, we might just as
well "leave the ComboBox bugs be". Otherwise, we could fix ComboBox. I'll
volunteer to fix bugs on it, but obviously this wouldn't make sense if you throw
the whole thing out anyway. So, what's your opinion on that?

Comment 2

[Felix]

I know why ComboBox sometimes isn't wide enough, and why it's sometimes wider
than it should be, depending on the font being used.

The problem is that ComboBox tries to calculate the maximum length of the box by
going through all of the strings in the list and calculating the size based on
the current font.

HOWEVER: the font used to calculate the maximum length is wrong - it's always
the same one. The function where the font is calculated is called

static void
SetMaximumLabelSize(DtComboBoxPart *combo_p)

in DtWidgets/ComboBox.c

Later on, the font is assigned correctly. But is should have been assigned
already when the ComboBox size is calculated. I will plunge a bit deeper into
this.

Comment 3

[Paul MacQuiddy]

Old bug, old codebase. Marking won't fix. Re-open if I am incorrect.

Comment 4

[yuli]

check

Comment 5

[radhika]

Attachment: ertertert

Comment 6

[Keiichi Matsumura]

Attachment: Screenshot

Comment 8

[Shiva]

I am nnot fixing this bug

Comment 9

[cafbot (PoC: ggrisco)]

Observed on: 

Device: evice
Moz BuildID:20140416000203
B2G Version: 1.4
Gecko Version: 30.0a2
Gaia:  http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=441c4bcd8ac4f8c01a9bc5a2f8d64eaa87844803
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=76bb0d325975cba44a3b61ffd7e21babfc706e4e