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

docs(Grid): 更新Grid demo #663

Merged
merged 1 commit into from
Mar 27, 2023
Merged

docs(Grid): 更新Grid demo #663

merged 1 commit into from
Mar 27, 2023

Conversation

wyj580231
Copy link
Collaborator

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 24, 2023

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

🕐 Build time: 294.483s

🤖 By surge-preview

@@ -35,7 +35,7 @@
<grid items="{{itemsCustom}}" onTap="handleTapItem" columns="{{5}}">
<view slot="icon" slot-scope="props">
<badge a:if="{{ props.value.tag }}" offsetX="-10px" type="text" text="{{ props.value.tag }}"><image src="{{ props.value.icon }}" style="width:44px;height:44px"/></badge>
<image a:else src="{{ props.value.icon }}" style="width:50px;height:50px"/>
<image a:else src="{{ props.value.icon }}" style="width:44px;height:44px"/>
</view>
<view slot-scope="props" slot="title">第{{ props.index + 1}}项</view>
<view slot-scope="props" slot="description">描述{{ props.index + 1}}</view>
Copy link
Contributor

Choose a reason for hiding this comment

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

这段代码看起来是在一个小程序中使用的,主要是对网格组件的图标进行了修改。从代码中可以看出以下改动:

  • 将图标默认大小从50x50px调整为44x44px;
  • 在显示图标时增加了一个标记属性。

这些改动并没有涉及到任何潜在的风险或错误。至于改进建议,可以考虑增加更详细的注释或文档,以便将来的开发者更好地理解该代码的作用和实现方式。

@jc9702507 jc9702507 merged commit 397f724 into master Mar 27, 2023
@github-actions github-actions bot mentioned this pull request Apr 3, 2023
@DiamondYuan DiamondYuan deleted the docs/Grid_0324 branch October 27, 2023 06:39
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