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

添加了导航栏logo和额外设置 #795

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

lingrottin
Copy link

总览

该修改涉及以下方面:

  • 添加了导航栏的 Logo
  • 在配置文件中添加了 Typed.js 中 autoInsertCss 选项的设置

分述

导航栏 Logo

在顶部导航栏 div.navbar-brand 内文字前添加了一个 <img> 标签。

效果

image

关于“单位自带”

因各种设备拥有繁复的各种 dpi、屏幕尺寸与比例,Logo 图片的大小不好以非相对单位 px 把握,本人使用 rem 单位配置 Logo 图片。而考虑到 px 以及其他单位确有用处,且主题用户大多未接触过 rem 单位,因此,依我拙见,此处单位应自带。

关于 CSS

在 CSS 中直接用 Stylus 变量而不是 CSS 自带的 var() 是受本人技术所限,无法寻找到定义 CSS 变量的文件,因而直接采用 Stylus 变量。
标签.类名 的选择器形式仅个人习惯,在 pr 时实无兴趣重新更改提交。
见谅。

关于 autoInsertCss

使 Typed.js 不自动插入 CSS 代码后,即可使光标不闪烁。
经本人测试,不会对采用 Fluid 的页面产生任何 除光标不闪烁外 的效果。

Demo

以上二修改项的效果均可在 [我的博客]([备用链接]) 看到。


(因为错误提交了 merge 到 master 的请求,重新提交了一遍)
辛苦了。

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.

None yet

1 participant