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

fix: the issue of the menu icon not being vertical center #569

Merged
merged 5 commits into from
Oct 4, 2024

Conversation

LIlGG
Copy link
Owner

@LIlGG LIlGG commented Aug 23, 2024

What type of PR is this?

/kind bug

What this PR does / why we need it:

解决导航栏图标未能垂直居中的问题

How to test it?

测试菜单图标是否垂直居中。
测试其他所有图标是否垂直居中。
测试是否会对界面其他元素有影响。

Which issue(s) this PR fixes:

Fixes #554

Does this PR introduce a user-facing change?

解决导航栏图标未能垂直居中的问题

@LIlGG
Copy link
Owner Author

LIlGG commented Aug 23, 2024

hi, @ShiinaKin ,我将 .iconify 中的 top 取消了,并对所有使用 icon 的位置使用了 flex 的居中方式。有空时你可以验证一下是否符合需求。

另外,由于改变了父元素的 display,因此有可能会导致某些内容受到影响。

@ShiinaKin
Copy link
Contributor

image

seems good, 符合预期

src/css/common/architecture/base.css Outdated Show resolved Hide resolved
@ShiinaKin
Copy link
Contributor

@LIlGG 请问这个pr还有什么问题吗,不能发正式版的话,希望先发一个rc

@LIlGG
Copy link
Owner Author

LIlGG commented Sep 14, 2024

@LIlGG 请问这个pr还有什么问题吗,不能发正式版的话,希望先发一个rc

最近在忙于其他项目,所以没有对主题进行维护,待此 PR 合并后再修改一些其他问题后会发正式版本。

@ruibaby
Copy link
Collaborator

ruibaby commented Sep 14, 2024

@LIlGG 请问这个pr还有什么问题吗,不能发正式版的话,希望先发一个rc

最近在忙于其他项目,所以没有对主题进行维护,待此 PR 合并后再修改一些其他问题后会发正式版本。

什么项目

@LIlGG LIlGG merged commit 8a70bdd into main Oct 4, 2024
1 check passed
@LIlGG LIlGG deleted the fix/menu-items-center branch October 4, 2024 11:59
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