From 5dc65fffe057f5f3135e55de4c7abe804fc5baa0 Mon Sep 17 00:00:00 2001 From: David Godinez Date: Mon, 6 Jan 2025 12:12:21 -0700 Subject: [PATCH 1/2] fix: Update Spectrum Them for Missing ContextualHelpTrigger Icon --- .../src/theme/theme-spectrum/theme-spectrum-overrides.css | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css b/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css index 4a46015d1c..0801e872f4 100644 --- a/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css +++ b/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css @@ -9,6 +9,12 @@ label[class*='spectrum-'] { margin-bottom: 0; } +[class*='spectrum-Menu-itemGrid'] svg, +svg[class*='spectrum-Textfield-validationIcon'] { + /* set as border-box by reboot, but spectrum expects this to be content-box */ + box-sizing: content-box; +} + svg[class*='spectrum-Textfield-validationIcon'] { /* set as border-box by reboot, but spectrum expects this to be content-box */ box-sizing: content-box; From 03ea22c8501c960870096fec2cb76ca0d697d99e Mon Sep 17 00:00:00 2001 From: David Godinez Date: Fri, 10 Jan 2025 14:26:38 -0700 Subject: [PATCH 2/2] add PR url --- .../src/theme/theme-spectrum/theme-spectrum-overrides.css | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css b/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css index 0801e872f4..122fff13ba 100644 --- a/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css +++ b/packages/components/src/theme/theme-spectrum/theme-spectrum-overrides.css @@ -9,6 +9,10 @@ label[class*='spectrum-'] { margin-bottom: 0; } +/* +* This workaround can be removed when this fix is pulled in +* https://github.com/adobe/react-spectrum/issues/7571 +*/ [class*='spectrum-Menu-itemGrid'] svg, svg[class*='spectrum-Textfield-validationIcon'] { /* set as border-box by reboot, but spectrum expects this to be content-box */