Smokey Ardisson (offline for a while; not following bugs - do not email) Reporter
	Description • 17 years ago

Since bug 381539, select widgets now have erroneous padding, as noted by smorgan in bug 381539 comment 1:

1) At the top and bottom of the list items and the scroll well
2) On the left and right of the list items

There's also an erroneous bevel/border (most noticeable at the bottom of the list); that might be bug 382040 or bug 382039, or it might be something else (I can file a separate bug if needed).

Note that these are regressions at least for Camino (except Camino had an erroneous 1 px padding at the top of the list that smfr was never able to eliminate).

	Smokey Ardisson (offline for a while; not following bugs - do not email) Reporter
	Comment 1 • 17 years ago

Today's Minefield, Safari, Camino 1.5

	Josh Aas
	Comment 3 • 17 years ago

Part of this problem is that listboxes draw outside of their frame because HITheme wants the content box, not the box for the whole control. This patch fixes that problem, but we still have a weird clipping problem remaining.

	Josh Aas
	Comment 4 • 17 years ago

fix v1.0 landed on trunk, leaving this open for the clipping issue

	Josh Aas
	Comment 5 • 17 years ago

Turns out the rest of this bug is cross-platform. I can see this exact same rendering problem with themed listboxes on Windows. There is a one-pixel-wide white lining drawn over the inside edge of the control. It obscures the left edge of the options and the right edge of the scrollbar and the top and bottom of both. I find it a little harder to see on Windows, but it is definitely there.

	Boris Zbarsky [:bzbarsky]
	Comment 7 • 17 years ago

So.. this worksforme on Linux.  Do you still see the problem if you disable native theming (using "-moz-appearance: none")?  Note that on Linux the white space is ONLY there if -moz-appearance is "listbox", and it does not overlap the option focus rect.  If I remove the -moz-appearance styling the white line goes away altogether.

This makes me think that this is at least partly a widget problem (esp. given it working on Linux).  In particular, I would start by looking at the padding and borders your theme code is giving the <select>.

As for the blocker, the initial blocking+ was for the bug as originally reported.  It's not clear to me that it applies to the remaining issue.  Especially since the Windows behavior has been as it is for years now, I bet.

	Josh Aas
	Comment 8 • 17 years ago

GTK uses a different setup for listboxes - for example, it does not specify any border or padding (afaict, from reading through the code). Linux is the one impl that doesn't show this bug and I imagine there are any number of reasons that might be true. Can you debug on Windows or Mac?

	Josh Aas
	Comment 9 • 17 years ago

On trunk linux with the native control (-moz-appearance: listbox) there is no white space between the options and the border of the control. I'm not sure what you're seeing. That white border not existing is probably part of the reason the bug doesn't show up on Linux.

I don't see this bug with Firefox 2 on Windows.

If I do -moz-appearance: none I don't see the bug on any platform, but like GTK that could be for any number of reasons. The border/padding is all different.

It would be best to debug this on a platform that does have the problem I think. I tried a bunch of native theme tweaking on Mac OS X and I can't get rid of the cropping.

	Boris Zbarsky [:bzbarsky]
	Comment 10 • 17 years ago

> there is no white space between the options and the border of the control.

There is over here.  Might depend on your GTK theme.

> I don't see this bug with Firefox 2 on Windows.

Can you find a regression range?

> Can you debug on Windows or Mac?

On Windows, no.  On Mac, maybe, but not until I get back in mid-September.  Even then I wouldn't be able to spend more than a few hours a week on it.

	Håkan Waara
	Comment 11 • 17 years ago

The remaining 1px problem is hardly a major bug.

	Robert O'Callahan (:roc) (email my personal email if necessary)
	Comment 12 • 17 years ago

I think I don't see this on Mac anymore, but I can't be sure because I don't know what testcase Josh used to produce comment #6. Anyway as Håkan said, this is not blocking anymore.

	Smokey Ardisson (offline for a while; not following bugs - do not email) Reporter
	Comment 13 • 17 years ago

I still see my comment 0 point 1 on my trunk build from last night; there's 1px of yellow between the top of the highlight (when the first item is selected) and top of the box (or bottom of the top border of the box, or however you want to describe it), and 1px of yellow between the bottom of the highlight (when the last item is selected) and the bottom of the box.  

You'll probably want to click elsewhere on the page to clear the nasty dotted focus ring when changing selections, since the ring+blue highlight+light yellow background+white bottom border makes the yellow harder to see there--or have a look with Pixie.

	philippe (part-time)
	Comment 14 • 17 years ago

this testcase makes it -maybe- easier to see the remaining issue

	philippe (part-time)
	Comment 15 • 17 years ago

(In reply to comment #14)
> Created an attachment (id=294003) [details]
> testcase
> 
> this testcase makes it -maybe- easier to see the remaining issue


When the top or bottom list-item is selected (focussed) a 1px tall white space (or coloured space, if the select has a background applied) before or after the selected item is visible.
This comes from this rule in forms.css:
select[size],
select[multiple],
select[size][multiple] {
  /* Different alignment and padding for listbox vs combobox */
  vertical-align: text-bottom;
  padding: 1px 0 1px 0;
  -moz-appearance: listbox;
}

That strip of space does not exist in WebKit/Safari (it does exist in Fx 2/Gecko 1.8.1, though). I don't have a windows or linux box at hand to verify what happens there.

	Boris Zbarsky [:bzbarsky]
	Comment 16 • 17 years ago

I believe that padding is there quite on purpose; you might want to look at the checkin history for the reasons for it, if any....  Also note that this is not what the bug was about to start with as far as I can tell (explicitly mentioned as being there in comment 0).

	Smokey Ardisson (offline for a while; not following bugs - do not email) Reporter
	Comment 17 • 17 years ago

Fair enough; I got confused over what comment 5 et seq. were talking about and then didn't re-read my comment 0 closely enough :(

Let's call this one FIXED, then, and we can open a new box on eliminating that spurious 1px at least on Mac OS X, where it looks out-of-place.

	Cork
	Comment 18 • 16 years ago

Is this bug only for multiple select objects?

I'm having problem with padding in a select box adds white space around the drop-down button. I would expect it to be added inside the box as in a multiple select box, and the drop-down button to be stretched (as with width/height).

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090129 Shiretoko/3.1b3pre

	Boris Zbarsky [:bzbarsky]
	Comment 19 • 16 years ago

This bug is only about listboxes, and only with the default styling.  If you have an issue with a combobox when certain styles are applied, please file a new bug if there isn't a bug covering it already.  Note that application of CSS to form controls is undefined, for what it's worth.