Confirmed users
116
edits
AMO:Editors/EditorGuide/CommonThemeProblems: Difference between revisions
Jump to navigation
Jump to search
AMO:Editors/EditorGuide/CommonThemeProblems (view source)
Revision as of 01:30, 1 August 2012
, 1 August 2012→Identity box is missing padlock icons for secure sites
| Line 47: | Line 47: | ||
===Address Bar=== | ===Address Bar=== | ||
====Identity box is missing padlock icons for secure sites==== | ====Identity box is missing padlock icons for secure sites==== | ||
In FF14+ the identity box has been significantly changed and the favicon has been replaced by an icon that indicates the website's status. In the case of secure sites the icon MUST be a padlock. For more information please see: https://wiki.mozilla.org/AMO:Editors/EditorGuide/ThemeReviews#Identity_Boxes | In FF14+ the identity box has been significantly changed and the favicon has been replaced by an icon that indicates the website's status. In the case of secure sites the icon MUST be a padlock. To fix this issue, you need to copy the files chrome://browser/skin/identity-icons-generic.png, chrome://browser/skin/identity-icons-https.png and chrome://browser/skin/identity-icons-https-ev.png to your "browser" folder. You can modify/redesign these icons to match your theme provided appropriate padlock icons are used for secure sites but not on insecure sites. The following CSS rules also need to be copied to the proper location in the browser.css file and modified as necessary: | ||
<br>--- | |||
<br> /* page proxy icon */ | |||
<br> | |||
<br> #page-proxy-favicon { | |||
<br> width: 16px; | |||
<br> height: 16px; | |||
<br> margin: 1px 3px; | |||
<br> list-style-image: url(chrome://browser/skin/identity-icons-generic.png); | |||
<br> -moz-image-region: rect(0, 16px, 16px, 0); | |||
<br> } | |||
<br> | |||
<br> .verifiedDomain > #identity-box-inner > #page-proxy-stack > #page-proxy-favicon { | |||
<br> list-style-image: url(chrome://browser/skin/identity-icons-https.png); | |||
<br> } | |||
<br> | |||
<br> .verifiedIdentity > #identity-box-inner > #page-proxy-stack > #page-proxy-favicon { | |||
<br> list-style-image: url(chrome://browser/skin/identity-icons-https-ev.png); | |||
<br> } | |||
<br> | |||
<br> #identity-box:hover > #identity-box-inner > #page-proxy-stack > #page-proxy-favicon { | |||
<br> -moz-image-region: rect(0, 32px, 16px, 16px); | |||
<br> } | |||
<br> | |||
<br> #identity-box:hover:active > #identity-box-inner > #page-proxy-stack > #page-proxy-favicon, | |||
<br> #identity-box[open=true] > #identity-box-inner > #page-proxy-stack > #page-proxy-favicon { | |||
<br> -moz-image-region: rect(0, 48px, 16px, 32px); | |||
<br> } | |||
<br> | |||
<br> #page-proxy-favicon[pageproxystate="invalid"] { | |||
<br> opacity: 0.5; | |||
<br> } | |||
<br>--- | |||
<br>For more information on identity box policies please see: https://wiki.mozilla.org/AMO:Editors/EditorGuide/ThemeReviews#Identity_Boxes | |||
====No visual clue for disabled url bars==== | ====No visual clue for disabled url bars==== | ||
There needs to be a visual clue when URL bar is disabled. To test this please go to https://bug486395.bugzilla.mozilla.org/attachment.cgi?id=370495 using the default theme and activate the "click me" link. The URL bar of the resulting popup window is disabled. | There needs to be a visual clue when URL bar is disabled. To test this please go to https://bug486395.bugzilla.mozilla.org/attachment.cgi?id=370495 using the default theme and activate the "click me" link. The URL bar of the resulting popup window is disabled. | ||