Skip to content

docs: optimize homepage ui - #199

Open
max-zhang78 wants to merge 3 commits into
goravel:masterfrom
max-zhang78:fix/ui-optimization
Open

docs: optimize homepage ui#199
max-zhang78 wants to merge 3 commits into
goravel:masterfrom
max-zhang78:fix/ui-optimization

Conversation

@max-zhang78

@max-zhang78 max-zhang78 commented Aug 19, 2026

Copy link
Copy Markdown

📑 Description

✅ Checks

  • Added test cases for my code

@hwbrzzl

hwbrzzl commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

@max-zhang78 Thanks for the optimization. Could you paste some screenshots for the difference between before and after in the PR description?

@max-zhang78

Copy link
Copy Markdown
Author
ScreenShot_2026-08-20_110257_006

适配大屏,导航栏统一风格,搜索框靠右,因为大屏会拉变形,logo变色特效

ScreenShot_2026-08-20_110359_914

导航栏统一背景色,子菜单缩进,链接颜色改成和logo颜色近似色,统一ui风格

ScreenShot_2026-08-20_110445_722

拉宽中间布局,方便长代码展示,这个才是主要想改的地方

https://max-zhang78.github.io/docs/zh_CN/ 这个展示demo的,更换了logo测试一下效果,这个没有在pr里

@hwbrzzl hwbrzzl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, basically LGTM.

on:
workflow_dispatch:
push:
branches: [ master ] # 推送到master自动触发

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We want to trigger the deployment manually to avoid mistake.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the optimization, but the actions in this file are what we want, so it's unnecessary to modify this file.

Comment thread zh_CN/index.md

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you optimize the uz_UZ index.md as well?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

OK, I modified deploy.yml to test my own demo code repository. I will retract these on the branch and optimize the uz-UZ index. md, I forgot about it. I will submit it later

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.

2 participants