Fix B85-ZK-3800, icon's height changed after F100-ZK-5119

zkoss · Oct 2, 2023 · 60fd266 · 60fd266
1 parent 055ef99
commit 60fd266
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/zul/src/main/resources/web/zul/less/font/_core.less b/zul/src/main/resources/web/zul/less/font/_core.less
@@ -15,12 +15,12 @@
 .@{fa-css-prefix}-regular,
 .fab,
 .@{fa-css-prefix}-brands {
-  -moz-osx-font-smoothing: grayscale;
+  display: inline-block;
+  font: normal normal normal @fa-font-size-base/@fa-line-height-base FontAwesome; // shortening font declaration
+  font-size: inherit; // can't have font-size inherit on line above, so need to override
+  text-rendering: auto; // optimizelegibility throws things off #1094
   -webkit-font-smoothing: antialiased;
-  display: ~'var(--@{fa-css-prefix}-display, @{fa-display})';
-  font-style: normal;
-  font-variant: normal;
-  text-rendering: auto;
+  -moz-osx-font-smoothing: grayscale;
 }
 
 .fas,

diff --git a/zul/src/main/resources/web/zul/less/font/_variables.less b/zul/src/main/resources/web/zul/less/font/_variables.less
@@ -6,6 +6,8 @@
 @fa-style-family        : 'FontAwesome';
 
 @fa-display             : inline-block;
+@fa-font-size-base      : 14px;
+@fa-line-height-base    : 1;
 
 @fa-fw-width            : (20em / 16);
 @fa-inverse             : #fff;