493738 - The CSS rule for sanitize.xul is moved from skin/preferences/preferences.css to skin/sanitizeDialog.css

Status: VERIFIED FIXED

(Firefox :: Theme, defect)

Description

[AKIHIRO Misaki (a.k.a Kuden)]

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090519 Minefield/3.6a1pre {Build ID: 20090519050407}

sanitize.xul uses skin/sanitizeDialog.css now.
However, sanitize.xul also still uses skin/preferences/preferences.css. 
http://hg.mozilla.org/mozilla-central/annotate/38d50cc03a72/browser/base/content/sanitize.xul#l45

It is necessary to move the CSS rule that sanitize.xul uses from skin/preferences/preferences.css to skin/sanitizeDialog.css, and not to use skin/preferences/preferences.css.

Comment 1

[Drew Willcoxon :adw]

To be clear, the sanitize.xul referred to in comment 0 is browser/base/content/sanitize.xul (the CRH dialog), not browser/components/preferences/sanitize.xul (a separate dialog involved with the privacy pref pane).

On Pinstripe the CRH dialog does use the .prefWindow-dlgbuttons and description selectors from browser/preferences/preferences.css.  .prefWindow-dlgbuttons sets all the margins of the dialog's OK/cancel buttons.  (browser/sanitizeDialog.css overrides the buttons' -moz-margin-end.)  description applies to the warning text when the duration dropdown is set to clear everything.

So this bug could be invalid, but maybe a better solution would be to remove Pinstripe's small dependency on browser/preferences/preferences.css.  CRH doesn't really look or act like a prefwindow anyway.

Comment 2

[AKIHIRO Misaki (a.k.a Kuden)]

(In reply to comment #1)
> So this bug could be invalid, but maybe a better solution would be to remove
> Pinstripe's small dependency on browser/preferences/preferences.css.  CRH
> doesn't really look or act like a prefwindow anyway.

I do not know Pinstripe in detail...

Pinstripe selects 1 or 2:
1. Some rules are copied from browser/preferences/preferences.css to browser/sanitizeDialog.css. 
2. Some rules are moved from browser/preferences/preferences.css to global/preferences.css.

Winstripe and Gnomestripe set .prefWindow-dlgbuttons on global/preferences.css. 
global/preferences.css is a style to general prefwindow, and browser/preferences/preferences.css is a style to Options (Prefwindow V) and the child prefwindow.

Comment 3

[AKIHIRO Misaki (a.k.a Kuden)]

Bug 504300 was fixed. 
The problem of comment #3 on Pinstripe (Firelight) was solved. 

Basically, when this line is deleted, this bug will be fixed.
http://hg.mozilla.org/mozilla-central/annotate/38bc9063e689/browser/base/content/sanitize.xul#l45

There is an anxiety of causing regression by the development in the future if this bug is not fixed. 
Because a needless element of browser/sanitizeDialog.css remains.

Comment 4

[Dão Gottwald [:dao]]

Kuden, would you like to create a patch for this? I could review it.

Comment 5

[AKIHIRO Misaki (a.k.a Kuden)]

(In reply to comment #4)
> Kuden, would you like to create a patch for this? I could review it.

I have not experienced the programming. 
And, I have not experienced CUI. 
Therefore, it is difficult for me to make the patch...

Comment 6

[(no longer active)]

Attachment: No-brainer :-)

Comment 7

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/02f8bf10f441

Comment 8

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

Verified fixed by code-checkin.