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

Update css-style-guide.md #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

rosieli
Copy link

@rosieli rosieli commented Jun 2, 2018

根据我们项目更新了样式规范

根据我们项目更新了样式规范
@otakustay
Copy link
Member

你更新的规范与现行规范严重不符,是出于什么考虑需要更新此处的规范?

@rosieli
Copy link
Author

rosieli commented Jun 5, 2018

不好意思,眼花看错了项目,不小心提交了,不过有几点项目规范也可以考究一下
1)模块与模块之间用明显分割线划分,并注明哪个模块
2)ID 不能和样式挂钩,只用于 js ,样式统一在 class
3)如果颜色值在项目中多次出现,可提取出变量引用,变量的命名可以参考 bootstrap

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