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: header & copyright margin, jetbrains mono font, missing message app image #155

Merged
merged 5 commits into from
Dec 5, 2023
Merged

fix: header & copyright margin, jetbrains mono font, missing message app image #155

merged 5 commits into from
Dec 5, 2023

Conversation

zhuozhiyongde
Copy link
Collaborator

@zhuozhiyongde zhuozhiyongde commented Nov 13, 2023

Description

增加了 Copyright 版权信息同正文的间距,便于区分。

移除了 Header 的左边距,因为这在小尺寸屏幕下会导致正文层叠可见。(对于右边距,因为是滚动条的空间所以无所谓,不过我发现好像 bilibili 的滚动条是悬浮不影响页面尺寸的(Safari/Chrome),不知道是否可以借鉴一下。

Cleanshot-2023-11-13-at-10 08 48

引入了 JetBrains-Mono 的 Google Font API,便于用户使用。

发现信息图片缺失(不知道在那次更新中没了还是最开始就没有),已添加。

Linked Issues

Additional context

目前右侧工具栏也有层叠问题,但是不太清楚怎么修复。

image

Copy link

vercel bot commented Nov 13, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @Innei on Vercel.

@Innei first needs to authorize it.

@zhuozhiyongde zhuozhiyongde changed the title fix: header & copyright margin, jetbrains mono font fix: header & copyright margin, jetbrains mono font, missing message app image Nov 13, 2023
Copy link

vercel bot commented Nov 13, 2023

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

Name Status Preview Comments Updated (UTC)
shiro ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 13, 2023 10:16am

Innei

This comment was marked as outdated.

@zhuozhiyongde
Copy link
Collaborator Author

那文字溢出有什么解决方法吗(

@Innei
Copy link
Owner

Innei commented Nov 13, 2023

那文字溢出有什么解决方法吗(

侧栏图标么,页面滚动到底是会这样的,因为向上偏移了 100%

@zhuozhiyongde
Copy link
Collaborator Author

那文字溢出有什么解决方法吗(

侧栏图标么,页面滚动到底是会这样的,因为向上偏移了 100%

不,我指的是标题栏Header左侧哪里

tailwind.config.ts Outdated Show resolved Hide resolved
@zhuozhiyongde
Copy link
Collaborator Author

@Innei 已经加回来了

@Innei Innei merged commit 2679e7b into Innei:main Dec 5, 2023
1 of 2 checks passed
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