In T364935#9828510, @Catrope wrote:I was confused about this at first, because I thought you could just pass a non-standard icon size into the icon mixin. But when I try that, I get:
load.php?lang=en&mod…22&version=1bu2z:31 Less_Exception_Compiler: variable @calculated-min-size-icon is undefined in file /var/www/html/w/resources/lib/codex/mixins/css-icon.less in css-icon.less on line 44, column 26 42| // This ensures that the icon will never appear smaller than @calculated-min-size-icon. 43| // Escape the max() call to prevent older Less versions from trying to do the max() calculation at compile time. 44| background-size: calc( ~'max( @{param-size-icon}, @{calculated-min-size-icon} )' );Now that we use rems, I wonder if we can remove the size-icon -> min-size-icon mapping, and instead just derive the min-size from the size by multiplying the size token by 16 and changing the unit from rem to px. Or alternatively, we could keep the mapping for known values, and do the math for custom values, or use the smallest min-size for custom values, or something like that.
The css-icon mixin should not throw an error when a non-standard size is passed in.