Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(Steps): delete empty axml onTap #711

Merged
merged 1 commit into from
Apr 27, 2023
Merged

fix(Steps): delete empty axml onTap #711

merged 1 commit into from
Apr 27, 2023

Conversation

wyj580231
Copy link
Collaborator

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 26, 2023

🎊 PR Preview ffd9b78 has been successfully built and deployed to https://ant-design-ant-design-mini-preview-pr-711.surge.sh

🕐 Build time: 264.074s

🤖 By surge-preview

@@ -1,6 +1,6 @@
<import-sjs from="./index.sjs" name="utils" />

<view class="ant-steps ant-steps-{{direction}} {{className ? className : ''}}" style="{{style}}" onTap="onChange">
<view class="ant-steps ant-steps-{{direction}} {{className ? className : ''}}" style="{{style}}">
<view a:for="{{items}}"
class="ant-steps-item ant-steps-item-{{direction}}
{{index < current? 'ant-steps-item-finish' : ''}}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

该代码补丁中的改动是将原有的onTap事件删除了,同时步骤条完成的状态ant-steps-item-finish被存储在样式中,用于确定当前步骤是否完成。代码风格看起来整洁清晰,但如果需要全面评估代码执行安全性和可读性,则需要更多上下文信息。

@codecov
Copy link

codecov bot commented Apr 26, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (be07e6f) 24.02% compared to head (ffd9b78) 24.02%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #711   +/-   ##
=======================================
  Coverage   24.02%   24.02%           
=======================================
  Files         158      158           
  Lines        3334     3334           
  Branches      875      875           
=======================================
  Hits          801      801           
  Misses       2202     2202           
  Partials      331      331           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jc9702507 jc9702507 merged commit 80ae8db into master Apr 27, 2023
@jc9702507
Copy link
Collaborator

#710

@github-actions github-actions bot mentioned this pull request May 9, 2023
@DiamondYuan DiamondYuan deleted the fix/Steps_0426 branch October 27, 2023 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants