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

optimize history card #566

Closed
wants to merge 2 commits into from
Closed

Conversation

xxfttkx
Copy link
Contributor

@xxfttkx xxfttkx commented Jan 6, 2025

优化历史记录中的卡片,防止按钮盖住番剧名
image
感觉这个按钮的正方形阴影和背景不同挺不自然的,查了一下,好像是PopupMenuButton的问题,我找不到方法解决

@Predidit
Copy link
Owner

Predidit commented Jan 6, 2025

我不能合并这个PR,除非这个问题得到解决

现在使用 Stack 堆叠好像就是为了规避这一问题

@xxfttkx
Copy link
Contributor Author

xxfttkx commented Jan 6, 2025

先改成draft了,我再想想有没有方法解决这个问题

@xxfttkx xxfttkx marked this pull request as draft January 6, 2025 12:40
@ErBWs
Copy link
Contributor

ErBWs commented Jan 6, 2025

我不能合并这个PR,除非这个问题得到解决

现在使用 Stack 堆叠好像就是为了规避这一问题

这个问题其实很好解决,使用 icon 代替 child 构建 PopupMenuButton,

image

@ErBWs ErBWs mentioned this pull request Jan 6, 2025
@xxfttkx xxfttkx closed this Jan 6, 2025
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.

3 participants