640421 - dropdown marker in urlbar is huge on Linux

Status: RESOLVED FIXED

(SeaMonkey :: Location Bar, defect)

Description

[Robert Kaiser]

In bug 401417, Neil and me both noticed that the urlbar and searchbar are vastly different in height on Linux due to an overly huge dropdown button in the urlbar. We might want to fix this in a way to make this one smaller.

Comment 1

[[e-mail]]

Attachment: Proposed patch

We're actually using the toolkit CSS, which needs enablehistory="true" (rather than the old XPFE disablehistory="false") to display correctly on Linux. The first side-effect is that we can't give a custom border to the URLbar itself, although we never used it as -moz-appearance took priority. The second is that enablehistory="true" sets moz-appearance on the autocomplete-textbox-container  which makes it impossible to turn it yellow on a secure site, so I had to give the URLbar its own specific XBL content to achieve this.

Comment 2

[Stefan [:stefanh]]

Comment on attachment 518500 [details] [diff] [review]
Proposed patch

There's an issue when you've focused the url bar on a secure site. It looks like the bottom of the dropmarker obscures a small part of the focus ring.

Comment 3

[Stefan [:stefanh]]

Attachment: Screenshots

Screenshots, demonstrating the issue.

Comment 4

[[e-mail]]

What if you back out the Mac theme changes? I only changed them to be consistent with Linux.

Comment 5

[Stefan [:stefanh]]

(In reply to comment #4)
> What if you back out the Mac theme changes? I only changed them to be
> consistent with Linux.

Yeah, that looks better. I'll take a more detailed look tomorrow - I haven't looked at the other autocompletes.

Comment 6

[Stefan [:stefanh]]

(In reply to comment #1)
> The second is that
> enablehistory="true" sets moz-appearance on the autocomplete-textbox-container 
> which makes it impossible to turn it yellow on a secure site, so I had to give
> the URLbar its own specific XBL content to achieve this.

Wouldn't "-moz-appearance: none" be better than adding all the xbl content?

Comment 7

[[e-mail]]

(In reply to comment #6)
> (In reply to comment #1)
> > The second is that
> > enablehistory="true" sets moz-appearance on the autocomplete-textbox-container 
> > which makes it impossible to turn it yellow on a secure site, so I had to give
> > the URLbar its own specific XBL content to achieve this.
> Wouldn't "-moz-appearance: none" be better than adding all the xbl content?
No, because on a Linux enablehistory autocomplete it's what draws the border.

Comment 8

[Stefan [:stefanh]]

(In reply to comment #7)
> > Wouldn't "-moz-appearance: none" be better than adding all the xbl content?
> No, because on a Linux enablehistory autocomplete it's what draws the border.

Aha, OK.

Comment 9

[Stefan [:stefanh]]

Comment on attachment 518500 [details] [diff] [review]
Proposed patch

This is fine on Mac default if you revert the mac/navigator/navigator.css changes.

Comment 10

[[e-mail]]

Attachment: Potential alternate mac styling

The previous patch went wrong because it applied the colour to an element contained inside a padded element. This should fix that by moving the padding.

This just replaces the hunk for mac's navigator.css from attachment 518500 [details] [diff] [review].

Comment 11

[Stefan [:stefanh]]

Comment on attachment 518780 [details] [diff] [review]
Potential alternate mac styling

Sorry, this won't work - it creates a white line under the dropmarker. See the screnshot in the next comment.

Comment 12

[Stefan [:stefanh]]

Attachment: Screenshot

Comment 13

[Stefan [:stefanh]]

Attachment: Screenshot - when focused

Comment 14

[[e-mail]]

(In reply to comment #11)
> Sorry, this won't work - it creates a white line under the dropmarker. See the
> screnshot in the next comment.
Bah, why does the Mac have to be so weird :-( I give up.

Comment 15

[Ian Neal]

Comment on attachment 518500 [details] [diff] [review]
Proposed patch

r=me with linux css changes only

Comment 16

[[e-mail]]

Pushed changeset 64392a2ec60c to comm-central.

Comment 17

[Jens Hatlak (:InvisibleSmiley)]

(In reply to comment #16)
> Pushed changeset 64392a2ec60c to comm-central.

FTR: You botched the bug number; maybe add a note to bug 420421?

Comment 18

[Stefan [:stefanh]]

Comment on attachment 518780 [details] [diff] [review]
Potential alternate mac styling

Clearing out obsolete request.