548626 - Placeholder attribute text should be italicized on Windows Vista and 7

Status: NEW

(Core :: Layout: Form Controls, defect)

Description

[Alex Faaborg [:faaborg] (Firefox UX)]

Spin off bug from bug 457800, the self describing text in the search field should be italicized to match the native appearance of self describing fields in Windows Vista and 7.

Comment 1

[(no longer active)]

I think an easy way of doing this would be using :-moz-system-metric(windows-default-theme) and :-moz-system-metric(windows-classic).

Comment 2

[Alex Faaborg [:faaborg] (Firefox UX)]

Wouldn't that trigger on Luna on XP though?  Not sure how search-box changes the style for Vista and 7.

Comment 3

[Mounir Lamouri (:mounir)]

Attachment: Patch v0.1

This patch may fix the bug. I have to test on Windows XP/Vista/7 to be sure it behaves correctly.
At least, it behaves correctly on GNU/Linux ;)

Comment 4

[Dão Gottwald [:dao]]

Comment on attachment 429596 [details] [diff] [review]
Patch v0.1

This is going to affect XP.

Comment 5

[(no longer active)]

(In reply to comment #4)
> (From update of attachment 429596 [details] [diff] [review])
> This is going to affect XP.

Hmm, yes, that's right.  Dao, do we have any way to style html.css level things per platform?

Comment 6

[Emil Ivanov]

Just noticed the summary that is for Windows only but the bug is assigned to OS X... So if this is Windows only, the platform have to be changed right?

Comment 7

[d]

Perhaps we need :-moz-system-metric(windows-5.0), :-moz-system-metric(windows-6.0) and :-moz-system-metric(windows-6.1) selectors? Would that be a good solution?

(5.0 is XP, 6.0 is Vista and 6.1 is Windows 7, based on the official version numbers. We could also consider creating selectors for other OS's and their versions in the same way, if we do this for Windows.)

Comment 8

[Mounir Lamouri (:mounir)]

I'm not really knowledgeable in this domain but probably adding :-moz-system-metric(windows-aero) would be enough ? We can even imagine this selector may be able to know if we are using XP or Aero theme on Windows.

Comment 9

[Dão Gottwald [:dao]]

We can just do this for chrome in toolkit/themes/winstripe/ once we have the placeholder pseudo class/element selector. I don't think placeholder text needs to be italicized by default for web content.

Comment 10

[Alex Faaborg [:faaborg] (Firefox UX)]

>I don't think placeholder text needs to be italicized by default for web content.

That's the purpose of the bug, external consistency with the OS.