1088578 - Replace rgba(...,0) and hsla(...,0) with 'transparent' in CSS gradients

Status: RESOLVED FIXED

(Firefox :: Theme, defect)

Description

[Dão Gottwald [:dao]]

As a workaround to bug 591600, we've been using code like this:

> linear-gradient(rgba(255,255,255,0.3), rgba(255,255,255,0));

instead of:

> linear-gradient(rgba(255,255,255,0.3), transparent);

Now that bug 591600 is fixed, we can use the latter.

Useful links:

http://mxr.mozilla.org/mozilla-central/search?string=%2C+rgba&find=themes&filter=0\%29

http://mxr.mozilla.org/mozilla-central/search?string=%2C+hsla&find=themes&filter=0\%29

Comment 1

[Sparsh Paliwal [:discoman]]

Please can i take up this.This would be my first one to work upon.

Comment 2

[Dão Gottwald [:dao]]

Sure, go ahead. Let me know if you have questions.

Comment 3

[Sparsh Paliwal [:discoman]]

Attachment: bug1088578.diff

I have replaced all the hsla and rgba with alpha(a) as 0 to transparent.

Comment 4

[Dão Gottwald [:dao]]

Comment on attachment 8511473 [details] [diff] [review]
bug1088578.diff

>--- a/browser/themes/linux/browser.css
>+++ b/browser/themes/linux/browser.css

> .chat-titlebar {
>   background-color: #d9d9d9;
>-  background-image: linear-gradient(@toolbarHighlight@, rgba(255,255,255,0));
>+  background-image: linear-gradient(@toolbarHighlight@,transparent);
> }

please keep the space after the comma

> .chatbar-button {
>   -moz-appearance: none;
>   background-color: #d9d9d9;
>-  background-image: linear-gradient(@toolbarHighlight@, rgba(255,255,255,0));
>+  background-image: linear-gradient(@toolbarHighlight@,transparent);
> }

ditto

>--- a/browser/themes/windows/browser.css
>+++ b/browser/themes/windows/browser.css

> #urlbar-stop-button:hover:active {
>-  background-image: radial-gradient(circle closest-side, hsla(5,100%,75%,.1), hsla(5,100%,75%,0));
>+  background-image: radial-gradient(circle closest-side, hsla(5,100%,75%,.1),transparent);
>   -moz-image-region: rect(28px, 28px, 42px, 14px);
> }

ditto

Looks good otherwise. Thanks!

Comment 5

[Sparsh Paliwal [:discoman]]

Attachment: bug1088578_whitespacefix.diff

i have made the changes directly to the patch file for the white space.

Comment 6

[Dão Gottwald [:dao]]

(In reply to Sparsh Paliwal from comment #5)
> Created attachment 8511595 [details] [diff] [review]
> bug1088578_whitespacefix.diff
> 
> i have made the changes directly to the patch file for the white space.

You've missed the first case (.chat-titlebar)...

Comment 7

[Sparsh Paliwal [:discoman]]

Attachment: bug1088578_whitespacefix2.diff

Apologies for the silly mistake committed.Thanks for the help and guidance......

Comment 8

[Dão Gottwald [:dao]]

Comment on attachment 8511663 [details] [diff] [review]
bug1088578_whitespacefix2.diff

Thanks!

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/fx-team/rev/3adeab7e472d

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/3adeab7e472d