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

style: 默认主题和暗黑主题样式精调 #845

Open
wants to merge 14 commits into
base: dev
Choose a base branch
from

Conversation

heshuyue
Copy link

@heshuyue heshuyue commented Jul 14, 2024

该 PR 调整默认主题和暗黑主题的样式

对应 issue: #800

具体修改内容都已在 commit 中说明。

默认主题

调整之前预览:
image

调整之后预览:
image

调整参考:

现阶段发现的问题:

  1. 网页全屏按钮失效(已解决)
  2. 建议增加回到顶部按钮(已解决)
  3. 悬浮菜单(cherry-floatmenu)的位置应该放在当前编辑行的上方,而不是右方,容易误点击(待优化)
  4. 跳转新窗口超链接的a::after元素图标不直观,会让人认为这只是一个普通链接,不是跳转新窗口的链接,同时不跳转新窗口的链接又没有a::after,就会造成超链接元素样式风格的割裂,普通超链接、蓝色文字以及跳转新窗口的超链接三者没有明显的区分,易造成混淆。应该更换一个更直观的图标。(待优化)例如:
    新窗口

暗黑主题

调整前预览:
image-20240831170313738

调整后预览:

image-20240905172706425

调整参考:

heshuyue added 7 commits July 11, 2024 17:48
工具栏的深蓝色与编辑区域中的大量文本的亮蓝色(链接与代码块)不统一
而且不好看
所以修改为了靛蓝
使得默认主题的风格更加一致
原始颜色与主题样式不统一
修改后清晰、统一、美观
使颜色与主题风格更加统一
修改后使按钮更清晰
以前的颜色比较浅,不直观
修改超链接hover时颜色与工具栏保持一致
超链接hover时增加下划线
移除目录超链接hover的下划线
修改目的:保持风格统一性,增强页面可读性和美观性
更改行内代码边框线大小
更加美观
Copy link

【预览链接】

@sunsonliu
Copy link
Collaborator

哇塞,你的【调整参考】真滴厉害,可以看出非常用心了,点赞~~

对于发现的问题,我尝试回复下:

  1. 网页全屏按钮是正常的,只不过在默认demo里因为已经是全屏状态了,所以没效果。若想体验,可以在这里体验全屏按钮功能
  2. 由于已经有了右侧悬浮目录,所以“回到顶部”的功能反而没有那么急迫了,所以就没有排issue实现,不过确实可以考虑增加这么个按钮
  3. 悬浮菜单位置确实挺尴尬的,不论放在哪都比较容易误触,所以大部分业务在引入cherry时确实都关掉了这个功能。若想彻底解决这个问题,可能还需要从长计议下
  4. 新页面打开的超链接图标确实不太好认,确实可以考虑统一设计、梳理下

最后,感谢pr,也期待有更多的pr~

@heshuyue
Copy link
Author

好的,非常感谢您的解答。

对于第1个问题,原来我们对于“全屏”的定义是不一样的哈哈哈,我自己理解的全屏是指网页铺满整个显示屏,也就是浏览器的标签页也会隐藏(效果和Windows处于浏览器时按下F11键相同),而你们的全屏只是铺满整个浏览器网页,那没问题了,哈哈,这里产生了歧义,不过,我见过的其他网站的“全屏”都是铺满整个显示屏,所以我觉得cherry-markdown的全屏应该叫做“网页全屏”更合适,不过这只是我的看法哈哈,肯定也有思考得不周到的地方。

对于第2个问题,我当时确实忽略了右侧悬浮目录,不够细心哈哈,那确实不是很有必要增加回到顶部按钮了。

最后,再次感谢您的细心解答😀@sunsonliu

@sunsonliu
Copy link
Collaborator

哈哈,那个“F11全屏”我懂,我们最开始实现的就是“F11全屏”,后来有人反馈这种全屏不好控制( #441 ),所以就改成当前这种形态了

@heshuyue
Copy link
Author

哈哈,那个“F11全屏”我懂,我们最开始实现的就是“F11全屏”,后来有人反馈这种全屏不好控制( #441 ),所以就改成当前这种形态了

原来是这样😀

Copy link

【预览链接】

Copy link

【预览链接】

Copy link

【预览链接】

此commit包含多处修改,全面修改原有的饱和色橙色主题,更改为对眼睛更友好的黑灰白浅色搭配主题,提升暗黑主题的适用度。
1 similar comment
@heshuyue heshuyue changed the title style: 默认主题样式精调 style: 默认主题和暗黑主题样式精调 Sep 5, 2024
增加目录的超链接下划线,让目录链接更加直观
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