monday-ui-style - npm Package Compare versions

Comparing version 0.1.159 to 0.1.160

package.json

 {
   "name": "monday-ui-style",
-  "version": "0.1.159",
+  "version": "0.1.160",
   "description": "Monday UI CSS Foundations",
@@ -5,0 +5,0 @@ "main": "dist/index.css",

stylelint-config/rules/use-defined-css-var-when-available/__tests__/index.test.js

@@ -98,3 +98,4 @@ const { lint } = require("stylelint");
 
-  it("warns for values that can be replaced with multiple CSS vars", async () => {
+  // We currently not supporting this use case after stop recommending use font size variables
+  it.skip("warns for values that can be replaced with multiple CSS vars", async () => {
     const {
@@ -121,3 +122,4 @@ results: [{ warnings }]
 
-  it("perform fixes when there are multiple var replacements, when specifying the useRecommendedFixes flag", async () => {
+  // We currently not supporting this use case after stop recommending use font size variables
+  it.skip("perform fixes when there are multiple var replacements, when specifying the useRecommendedFixes flag", async () => {
     const { results } = await lint({
@@ -124,0 +126,0 @@ files: path.resolve(__dirname, "./fixtures/contains-values-with-multiple-replacements.scss"),

stylelint-config/rules/use-defined-css-var-when-available/__tests__/props-to-allowed-vars.test.js

 const { parseMondayUiCss, getPropsToAllowedCssVars } = require("../parse-monday-css");
 const { PROPS_TO_ALLOWED_VARS } = require("../props-to-allowed-vars");
 
-const ALLOWED_CSS_VARS_WITHOUT_MAPPINGS = ["--font-line-height-30", "--font-line-height-20", "--font-size-10"];
+const ALLOWED_CSS_VARS_PREFIX_WITHOUT_MAPPING = [
+  "--font-h",
+  "--font-text",
+  "--font-weight",
+  "--font-size",
+  "--font-line-height",
+  "--letter-spacing"
+];
 
@@ -31,3 +38,3 @@ describe("props-to-allowed-vars", () => {
 
-        if (!ALLOWED_CSS_VARS_WITHOUT_MAPPINGS.includes(varFromGeneratedCss)) {
+        if (!ALLOWED_CSS_VARS_PREFIX_WITHOUT_MAPPING.some(prefix => varFromGeneratedCss.startsWith(prefix))) {
           // If this fails, a CSS var was added without mapping it to recommended CSS props.
@@ -34,0 +41,0 @@ // Without this mapping, we won't be able to recommend the proper usage of the CSS var.

stylelint-config/rules/use-defined-css-var-when-available/props-to-allowed-vars.js

@@ -149,16 +149,2 @@ const SPACINGS = [
   ...mapPropsToAllowedVars(BORDER_STYLES_PROPS, BORDER_STYLES),
-  ...mapPropsToAllowedVars("line-height", FONT_LINE_HEIGHTS, [
-    "--font-line-height-subtext",
-    "--font-line-height-h1",
-    "--font-line-height-h2",
-    "--font-line-height-h4"
-  ]),
-  ...mapPropsToAllowedVars("font-weight", FONT_WEIGHTS),
-  ...mapPropsToAllowedVars("font-size", FONT_SIZES, [
-    "--font-size-h1",
-    "--font-size-h2",
-    "--font-size-h4",
-    "--font-size-h5",
-    "--font-size-general-label"
-  ]),
   ...mapPropsToAllowedVars(FONT_FAMILIES_PROPS, FONT_FAMILIES, ["--title-font-family"]),
@@ -165,0 +151,0 @@ ...mapPropsToAllowedVars(TIMING_FUNCTION_PROPS, ANIMATION_TIMING, ["--expand-animation-timing"]),

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

718248

increased by1.31%

Number of package files

340

increased by1.19%

Lines of code

4535

increased by1.66%

No dependency changes