From 67b32df6db93d929fc868c28ca1c815d5d10f618 Mon Sep 17 00:00:00 2001 From: Nidhi Manu Date: Tue, 13 Feb 2024 08:55:42 -0500 Subject: [PATCH 1/2] Revert "Change icons from divs to spans (#1857)" This reverts commit 2d3c39ecc8bafe53584fbc46baf7a5c4e0d15f89 because it causes some visual regressions in the HH theme. The HH Theme has styling that targets spans directly and changing the icon elements from divs to spans caused the icons to be inadverdently targeted by that styling. --- src/ui/sass/modules/_SearchBar.scss | 5 ----- src/ui/templates/icons/builtInIcon.hbs | 4 ++-- src/ui/templates/icons/icon.hbs | 4 ++-- src/ui/templates/icons/iconPartial.hbs | 4 ++-- src/ui/templates/icons/searchBarIcon.hbs | 24 +++++++++++----------- src/ui/templates/icons/voiceSearchIcon.hbs | 2 +- 6 files changed, 19 insertions(+), 24 deletions(-) diff --git a/src/ui/sass/modules/_SearchBar.scss b/src/ui/sass/modules/_SearchBar.scss index 650d76045..d3cf8879f 100644 --- a/src/ui/sass/modules/_SearchBar.scss +++ b/src/ui/sass/modules/_SearchBar.scss @@ -280,7 +280,6 @@ $searchbar-button-text-color-active: var(--yxt-searchbar-button-text-color-base) &-buttonImage .Icon svg, .Icon-image { - display: block; width: 2em; height: 2em; } @@ -288,10 +287,6 @@ $searchbar-button-text-color-active: var(--yxt-searchbar-button-text-color-base) &-instructions { display: none; } - - &-CustomLoadingImage { - display: block; - } } // If custom icons are supplied, the SVGs simply alternate opacity diff --git a/src/ui/templates/icons/builtInIcon.hbs b/src/ui/templates/icons/builtInIcon.hbs index 00b0c3b82..0355d648e 100644 --- a/src/ui/templates/icons/builtInIcon.hbs +++ b/src/ui/templates/icons/builtInIcon.hbs @@ -1,3 +1,3 @@ -