Skip to content
This repository was archived by the owner on Aug 12, 2025. It is now read-only.

Conversation

@ZaneL1u
Copy link

@ZaneL1u ZaneL1u commented Mar 10, 2022

before:
before

after:
after

@auto-assign auto-assign bot requested review from JohnNiang, LIlGG, guqing and ruibaby March 10, 2022 07:35
Copy link
Member

@JohnNiang JohnNiang left a comment

Choose a reason for hiding this comment

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

Thanks for your first contribution!

LGTM

@ZaneL1u
Copy link
Author

ZaneL1u commented Mar 10, 2022

啊,图太慢了,问题和效果可以看这里:
before: https://cdn.blog.athesoft.com/halo/before_1646897548095.gif
after: https://cdn.blog.athesoft.com/halo/after_1646897544786.gif

Copy link
Member

@ruibaby ruibaby left a comment

Choose a reason for hiding this comment

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

LGTM

@guqing guqing merged commit 5b17179 into halo-dev:master Mar 10, 2022
@ruibaby
Copy link
Member

ruibaby commented Mar 10, 2022

@Yorksh1re Hi,是否可以再看看其他地方需不需要改呢?

@ZaneL1u
Copy link
Author

ZaneL1u commented Mar 10, 2022

好的,我看下哈~

@ruibaby
Copy link
Member

ruibaby commented Mar 10, 2022

好的,我看下哈~

期待下次贡献。😀

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


zhenze seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@ruibaby ruibaby added this to the 1.5.x milestone Mar 10, 2022
@ruibaby ruibaby added the kind/feature Categorizes issue or PR as related to a new feature. label Mar 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

kind/feature Categorizes issue or PR as related to a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants