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

feat: Support skip to guide in simple mode #532

Merged
merged 14 commits into from
Mar 21, 2024

Conversation

shanchuan1
Copy link
Contributor

Copy link

changeset-bot bot commented Jan 24, 2024

🦋 Changeset detected

Latest commit: 6c71dfd

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@ant-design/web3-common Patch
@ant-design/web3 Patch
@ant-design/web3-assets Patch
@ant-design/web3-solana Patch
@ant-design/web3-wagmi Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Jan 24, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ant-design-web3 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 21, 2024 3:48am

Copy link

github-actions bot commented Jan 24, 2024

Copy link

codecov bot commented Jan 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (d374e55) to head (6c71dfd).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #532   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          650       650           
  Lines        17505     17558   +53     
  Branches      1071      1075    +4     
=========================================
+ Hits         17505     17558   +53     

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

@thinkasany
Copy link
Collaborator

需要覆盖一下单测
image
image

@yutingzhao1991
Copy link
Collaborator

image

感觉没对齐

@yutingzhao1991
Copy link
Collaborator

另外这个线应该是平齐两边的,高度是 70px(不过这个高度不一定对,我再找设计师确认下)

image

@yutingzhao1991
Copy link
Collaborator

另外 footer 也存在的时候如何显示,现在是怎么处理的?

@shanchuan1
Copy link
Contributor Author

shanchuan1 commented Jan 24, 2024

另外 footer 也存在的时候如何显示,现在是怎么处理的?

footer同时存在走footer逻辑,那么默认的
image
不展示

@yutingzhao1991
Copy link
Collaborator

另外 footer 也存在的时候如何显示,现在是怎么处理的?

footer同时存在走footer逻辑,那么默认的 image 不展示

感觉两个应该都保留,是不一样的用途。

@yutingzhao1991
Copy link
Collaborator

LearnMore 点击后应该显示:
image

@yutingzhao1991
Copy link
Collaborator

@wenqing993 文卿帮忙看看这个问题,看看下面这个两个元素在简单窗口模式下如何一起显示:

image

@wenqing993
Copy link

好的,我这边看一下

@yutingzhao1991
Copy link
Collaborator

yutingzhao1991 commented Jan 26, 2024

@shanchuan1 设计师那边这两天会有新的设计稿给到,稍微等一下

@shanchuan1
Copy link
Contributor Author

@shanchuan1 设计师那边这两天会有新的设计稿给到,稍微等一下

ok

@yutingzhao1991
Copy link
Collaborator

@shanchuan1 设计师那边这两天会有新的设计稿给到,稍微等一下

ok

设计稿好了,我在 issue 中回复了。

@LCJove
Copy link
Collaborator

LCJove commented Mar 6, 2024

@shanchuan1 同学这个pr你还有精力跟嘛,我可以帮忙一起看一下

packages/common/src/locale/zh_CN.ts Outdated Show resolved Hide resolved
packages/common/src/locale/en_US.ts Outdated Show resolved Hide resolved
packages/common/src/locale/en_US.ts Outdated Show resolved Hide resolved
@yutingzhao1991
Copy link
Collaborator

simplemode.bug.mov

@LCJove 又发现一个 bug,小窗下点击到 guide,然后调整为大窗口,这个时候多了一个返回的导航条,这种情况要考虑一下

@LCJove
Copy link
Collaborator

LCJove commented Mar 19, 2024

simplemode.bug.mov
@LCJove 又发现一个 bug,小窗下点击到 guide,然后调整为大窗口,这个时候多了一个返回的导航条,这种情况要考虑一下

已解决

@yutingzhao1991
Copy link
Collaborator

@LCJove 代码有冲突了

@LCJove LCJove merged commit cb0a288 into ant-design:main Mar 21, 2024
9 checks passed
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.

6 participants