From ea919b6e262d74a48b910a84604b400a2178c884 Mon Sep 17 00:00:00 2001
From: Ariella Eliassaf <ariella@codemagic.co.il>
Date: Tue, 27 Jun 2023 10:36:50 +0300
Subject: [PATCH] fix(pushbutton-element): incorrect colors for button gradient
 on safari when there are multiple buttons

close #153
---
 src/pushbutton-element.ts | 24 +++++++++++++-----------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/src/pushbutton-element.ts b/src/pushbutton-element.ts
index 68bff92..c0f1d61 100644
--- a/src/pushbutton-element.ts
+++ b/src/pushbutton-element.ts
@@ -33,10 +33,6 @@ export class PushbuttonElement extends LitElement {
         -moz-appearance: none;
       }
 
-      button:active .button-circle {
-        fill: url(#grad-down);
-      }
-
       .clickable-element {
         cursor: pointer;
       }
@@ -56,7 +52,7 @@ export class PushbuttonElement extends LitElement {
 
   render() {
     const { color, label } = this;
-    const buttonFill = this.pressed ? 'url(#grad-down)' : 'url(#grad-up)';
+    const buttonFill = this.pressed ? `url(#grad-down-${color})` : `url(#grad-up-${color})`;
 
     return html`
       <button
@@ -76,18 +72,24 @@ export class PushbuttonElement extends LitElement {
           xmlns="http://www.w3.org/2000/svg"
           xmlns:xlink="http://www.w3.org/1999/xlink"
         >
+          <style>
+            button:active .button-circle {
+              fill: url(#grad-down-${color});
+            }
+          </style>
           <defs>
-            <linearGradient id="grad-up" x1="0" x2="1" y1="0" y2="1">
+            <linearGradient id="grad-up-${color}" x1="0" x2="1" y1="0" y2="1">
+              <stop stop-color="#ffffff" offset="0" />
+              <stop stop-color="${color}" offset="0.3" />
+              <stop stop-color="${color}" offset="0.5" />
+              <stop offset="1" />
+            </linearGradient>
+            <linearGradient id="grad-down-${color}" x1="1" x2="0" y1="1" y2="0">
               <stop stop-color="#ffffff" offset="0" />
               <stop stop-color="${color}" offset="0.3" />
               <stop stop-color="${color}" offset="0.5" />
               <stop offset="1" />
             </linearGradient>
-            <linearGradient
-              id="grad-down"
-              xlink:href="#grad-up"
-              gradientTransform="rotate(180,0.5,0.5)"
-            ></linearGradient>
           </defs>
           <rect x="0" y="0" width="12" height="12" rx=".44" ry=".44" fill="#464646" />
           <rect x=".75" y=".75" width="10.5" height="10.5" rx=".211" ry=".211" fill="#eaeaea" />