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: add navbar #73

Merged
merged 32 commits into from
Mar 12, 2024
Merged

feat: add navbar #73

merged 32 commits into from
Mar 12, 2024

Conversation

xiaowangxiaowang256256
Copy link
Contributor

@xiaowangxiaowang256256 xiaowangxiaowang256256 commented Mar 10, 2024

#70
本次更新内容:
1、修复:调整AI 小助手图片,之前图片在黑暗模式下 有白色像素;
2、修复:Sponsor list 在手机模式下 表格溢出;
3、功能:添加导航条,便于后续 菜单添加与维护;

2d2a2e2fb080de70ca5fe550d82f38f
35de69444a2174c537d6e15b08ad377
image

Copy link

vercel bot commented Mar 10, 2024

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

Name Status Preview Comments Updated (UTC)
opensource-f2f-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 12, 2024 7:15am

@xiaowangxiaowang256256
Copy link
Contributor Author

@sxzz @LinuxSuRen 亲爱的小伙伴们,大家上午好。如果有空余时间,帮我 review 一下代码哈,有什么问题我及时整改哈

src/App.vue Outdated Show resolved Hide resolved
src/components/sponsors.vue Outdated Show resolved Hide resolved
Copy link
Member

@sxzz sxzz left a comment

Choose a reason for hiding this comment

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

We should also add an icon for menu list. Use heart with pink color for sponsor

@LinuxSuRen
Copy link
Member

We should also add an icon for menu list. Use heart with pink color for sponsor

Sounds good.

@xiaowangxiaowang256256
Copy link
Contributor Author

We should also add an icon for menu list. Use heart with pink color for sponsor

Sounds good.

Refer to this.

@xiaowangxiaowang256256
Copy link
Contributor Author

We should also add an icon for menu list. Use heart with pink color for sponsor

Have finished.

@xiaowangxiaowang256256
Copy link
Contributor Author

I want to use lg-visiable to control the element display, but it does not take effect, not clear the specific reason, the current can only use the media query way to solve the element display. Can you help me find out the reason
--- 我想使用 lg-visiable 来控制 元素的显示,但是没有生效,不太清楚具体的原因哈,当前只能使用的是 媒体查询的方式,来解决元素显示。你可以帮我查找一下原因么
image

@sxzz
Copy link
Member

sxzz commented Mar 12, 2024

默认所有元素都是显示 visible 的(visiable拼写有误),我们要的应该是 隐藏 hidden sm-hidden, md-hidden。不同断点可以查阅 https://tailwindcss.com/docs/responsive-design#using-custom-breakpoints

@xiaowangxiaowang256256
Copy link
Contributor Author

默认所有元素都是显示 visible 的(visiable拼写有误),我们要的应该是 隐藏 hidden sm-hidden, md-hidden。不同断点可以查阅 https://tailwindcss.com/docs/responsive-design#using-custom-breakpoints

1、嗯嗯 单词拼写有误了
2、我的需求是:将该元素 >640px 像素时显示,<=640px 时隐藏。当前我使用的是 useMediaQuery('(min-width: 640px)') 来查询屏幕,并控制显示。我再去研究研究 断点的使用方法哈

Copy link
Member

@sxzz sxzz left a comment

Choose a reason for hiding this comment

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

Refactored a lot for you

@LinuxSuRen
Copy link
Member

Conflicting files found.

@sxzz sxzz merged commit 0785d9d into opensource-f2f:main Mar 12, 2024
1 of 2 checks passed
@xiaowangxiaowang256256
Copy link
Contributor Author

Nice, navigation menu successfully launched ~

@LinuxSuRen
Copy link
Member

image

Looks great!

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