From 5ccf55cbd18eacce61dc776934cf96d1f9386b98 Mon Sep 17 00:00:00 2001 From: Danny Colin Date: Thu, 20 Oct 2022 14:22:17 -0400 Subject: [PATCH 1/2] Fix broken radio focus styling in light mode --- src/css/popup.css | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/src/css/popup.css b/src/css/popup.css index db7a71ee..5980894c 100644 --- a/src/css/popup.css +++ b/src/css/popup.css @@ -107,8 +107,9 @@ --toggle-on-bg-focus-color: #0060df; --toggle-on-border-color: #0060df; --toggle-on-handle-bg-color: #fff; - --usercontext-bg-hover-color: #e0e0e6; - --usercontext-bg-active-color: #e0e0e6; + --usercontext-bg-hover-color: #f0f0fa; + --usercontext-bg-focus-color: #e0e0e6; + --usercontext-bg-active-color: #cfcfd8; --moz-vpn-tout-shadow: 0 0 7px 0 #9498a25e; } @@ -1884,15 +1885,25 @@ manage things like container crud */ margin-inline: auto; } +/* When focusing the element add a thin blue highlight to match input fields. This gives a distinction to other selected radio items */ +.radio-choice > .radio-container > [type="radio"]:focus + label { + outline: solid 1px var(--input-border-focus-color); + -moz-outline-radius: 100%; +} + .radio-choice > .radio-container > [type="radio"]:checked + label { background-color: var(--usercontext-bg-focus-color); border-radius: 100%; } -/* When focusing the element add a thin blue highlight to match input fields. This gives a distinction to other selected radio items */ -.radio-choice > .radio-container > [type="radio"]:focus + label { - outline: solid 1px var(--input-border-focus-color); - -moz-outline-radius: 100%; +.radio-choice > .radio-container > [type="radio"]:checked + label:hover { + background-color: var(--usercontext-bg-hover-color); + border-radius: 100%; +} + +.radio-choice > .radio-container > [type="radio"]:checked + label:active { + background-color: var(--usercontext-bg-active-color); + border-radius: 100%; } .edit-container-panel fieldset { From c4b0823526e7a7a44360c1ab5f341338df57cfd4 Mon Sep 17 00:00:00 2001 From: Danny Colin Date: Thu, 20 Oct 2022 17:58:06 -0400 Subject: [PATCH 2/2] bump version to 8.1.1 --- package.json | 2 +- src/manifest.json | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/package.json b/package.json index 53958f17..1fdf1d48 100644 --- a/package.json +++ b/package.json @@ -2,7 +2,7 @@ "name": "testpilot-containers", "title": "Multi-Account Containers", "description": "Containers helps you keep all the parts of your online life contained in different tabs. Custom labels and color-coded tabs help keep different activities — like online shopping, travel planning, or checking work email — separate.", - "version": "8.1.0", + "version": "8.1.1", "author": "Andrea Marchesini, Luke Crouch, Lesley Norton, Kendall Werts, Maxx Crawford, Jonathan Kingston", "bugs": { "url": "https://github.com/mozilla/multi-account-containers/issues" diff --git a/src/manifest.json b/src/manifest.json index 247497fb..e0f9765b 100644 --- a/src/manifest.json +++ b/src/manifest.json @@ -1,7 +1,7 @@ { "manifest_version": 2, "name": "Firefox Multi-Account Containers", - "version": "8.1.0", + "version": "8.1.1", "incognito": "not_allowed", "description": "__MSG_extensionDescription__", "icons": {