fix: logical attribute calc css variable logic error

[Wxh16144]

fix: ant-design/ant-design#47652
fix: ant-design/ant-design#47707

[MadCcc]

cssinjs/src/transformers/legacyLogicalProperties.ts

Line 32 in b58b081

temp += item;

这里加个空格应该就可以

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.80%. Comparing base (907a22b) to head (6c8152d).

❗ Current head 6c8152d differs from pull request most recent head a79f96a. Consider uploading reports for the commit a79f96a to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #177      +/-   ##
==========================================
- Coverage   94.80%   94.80%   -0.01%     
==========================================
  Files          29       29              
  Lines        2619     2618       -1     
  Branches      394      393       -1     
==========================================
- Hits         2483     2482       -1     
  Misses        136      136              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[yoyo837]

为啥不都用数组push, 然后最终join(' ')呢？字符串玩着玩着容易玩出这种问题。

[afc163]

+1

[Wxh16144]

感觉这段代码已经比较精简了，没有可操作空间了

[yoyo837]

  let temp: string[] = [];
  let brackets = 0;
  return [
    splitStyle.reduce<string[]>((list, item) => {
      if (item.includes('(') || item.includes(')')) {
        const left = item.split('(').length - 1;
        const right = item.split(')').length - 1;
        brackets += left - right;
      }
      if (brackets >= 0) {
        temp.push(item);
      }
      if (brackets === 0) {
        list.push(temp.join(' '));
        temp = [];
      }
      return list;
    }, []),
    !!importantCells,
  ];

[afc163]

@yoyo837 可以直接用 suggestion 能力。

[yoyo837]

跨越很多行，还涉及一些未修改的行，这种情况我不会 suggestion

[Wxh16144]

push 了，再 review 一下吧

[yoyo837]

不省略括号好一些