301854 - Unable to select Directory Server from dropdown because it's too slim

Status: RESOLVED WORKSFORME

(Toolkit :: UI Widgets, defect)

Description

[Adam Guthrie]

1. Edit > Preferences > Composition > Addressing tab.

It's impossible to use the dropdown to select a Directory Server because the
dropdown widget is way too small. Clicking on it clearly activates the widget
(you can see the radio box turn gray), but it will not drop down.

Comment 1

[Adam Guthrie]

Attachment: screenshot of the dropdown widget

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

This should be a regression after removing the dropdown image from menulist. We
should set a height value so that also empty menulist will be shown correctly. I
think that I could come up with a patch within the next days.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Correct height of an empty menulist

We exactly need a height of 2.2em for an empty menulist. I tested with
different display resolutions (75dpi, 96dpi, 154dpi). All of them look fine.

Comment 4

[Benjamin Smedberg]

Comment on attachment 190325 [details] [diff] [review]
Correct height of an empty menulist

This seems wrong to me... why are we specifying the height of a menulist
element?

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 190325 [details] [diff] [review]
Correct height of an empty menulist

Clearing flag. See bug 301858 comment 4. We have to wait until bug 285076 will
be fixed.

Comment 6

[Adam Guthrie]

I still see this same issue after bug 285076 was checked in. It appears like
shown in the screenshot (but now it's shown as grayed out (disabled)) when one
doesn't have any servers added. Shouldn't it just be a blank dropdown in this
case? (Works fine when servers are added.)

Comment 7

[Adam Guthrie]

*** Bug 320721 has been marked as a duplicate of this bug. ***

Comment 8

[klonos]

I don't see this in Ubuntu 8.10 and Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1b3pre) Gecko/20090227 Lightning/1.0pre Shredder/3.0b2

Fixed??

Will try to install 3.1pre as well and report back.

Comment 9

[Worcester12345]

Just an FYI, but this seems OK in 60.7.0 (32-bit).

Comment 10

[Wayne Mery (:wsmwk)]

Thanks