fix(comp:icon): ellipsis icon not right

[sallerli1]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines
• Tests for the changes have been added/updated or not needed
• Docs and demo have been added/updated or not needed

What is the current behavior?

省略号图标错误

What is the new behavior?

将错误的省略号图标修改为 ellipsis-vertical，换回原来正确的图标

Other information

[cr-gpt]

with the review

1. There is no typos or syntax errors in the code patch, which is a good sign for code quality.

2. The new icon 'ellipsis-vertical' appears to be added correctly in the list of icons.

3. It would be nice if the code follows a consistent style and indentation.

4. It is good to have some basic tests on the code patch to make sure it works as expected.

[cr-gpt]

the code review

This code patch appears to be a valid SVG vector graphic. It can be used to generate a shape or pattern for use in web graphics. The code appears to be well written and correctly formatted, with appropriate indentation and spacing. No obvious bugs or risks are present.

However, it would be beneficial to do some additional testing on the code to ensure that it displays properly on all browsers, or if there are any compatibility issues. Additionally, it may be helpful to review the code for any unnecessary or redundant lines of code that can be removed in order to optimize performance and reduce file size.

[cr-gpt]

with a brief code review

The code patch above is used to replace an existing SVG element with a new SVG element. The new SVG element includes a defs element and a style element, which provides additional styling flexibility. It also includes a path element that defines the shape of the SVG.

From a code review perspective, this code should be tested to ensure that the new SVG element renders properly in the browser, and that the styling and shape of the element is correct. Additionally, it could be tested to ensure that the code is optimized for performance and does not cause any compatibility issues.

[idux-bot]

This preview will be available after the AzureCI is passed.

[cr-gpt]

with the code review:

1. The code looks to be related to SVG graphics. It appears that the patch is replacing an old SVG graphic with a new one. It is not clear what the purpose of the graphic is, but it should be tested to ensure that it renders correctly.

2. From a security perspective, it is important to check that the new graphic does not contain any malicious code. Additionally, any changes to the code should be tested to ensure that no unexpected behavior occurs.

3. The code should be checked for any potential performance issues. Any large or complex graphics should be optimized to ensure that they do not cause any performance degradation.

4. Generally speaking, the code looks to be well written and should not require any major changes. However, it could be improved by adding comments to explain the purpose of the code. This would make it easier for future developers to understand the code.

Overall, the code looks to be in good condition. With a few minor improvements, it should be ready for deployment.

[codecov]

Codecov Report

Merging #1522 (e69723d) into main (a781bb9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head e69723d differs from pull request most recent head 9e5f656. Consider uploading reports for the commit 9e5f656 to get more accurate results

@@           Coverage Diff           @@
##             main    #1522   +/-   ##
=======================================
  Coverage   92.77%   92.77%           
=======================================
  Files         332      332           
  Lines       30892    30897    +5     
  Branches     3555     3556    +1     
=======================================
+ Hits        28661    28666    +5     
  Misses       2231     2231           

Impacted Files	Coverage Δ
packages/components/icon/src/definitions.ts	100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.