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.

Sign up for GitHub

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

Jump to bottom

设置界面使用弹窗 #871

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

设置界面使用弹窗 #871

wants to merge 2 commits into from

Conversation

kangfenmao
Copy link
Collaborator

No description provided.

@kangfenmao
Copy link
Collaborator Author

  1. 需要提供一个设置选项支持弹窗方式显示设置界面（满足个性化需求）
  2. 背景模糊会是全局的，容易让人产生眩晕
  3. 弹窗高度需要高一些
  4. 弹窗菜单风格需要和界面其他地方风格保持一致（体验不会割裂）

image

@kangfenmao kangfenmao requested a review from teojs January 27, 2025 15:15
@teojs
Copy link
Collaborator

teojs commented Jan 28, 2025

风格方面，目前觉得整体风格偏向Web化，不像一个APP，所以做了一些调整，后续还会对整体UI进行重构。目前如果觉得割裂的话，就先不合并了吧，等后续UI重构一起，然后看看是否符合需求。

@kangfenmao
Copy link
Collaborator Author

可以在这个分支上继续弄，我们两个可以一起来开发

@kangfenmao
Copy link
Collaborator Author

最近有点忙，等我闲下来再处理这个合并

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@teojs teojs Awaiting requested review from teojs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kangfenmao @teojs