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

box-model rule not respecting box-sizing property #298

Closed
jstrimpel opened this issue Aug 23, 2012 · 1 comment
Closed

box-model rule not respecting box-sizing property #298

jstrimpel opened this issue Aug 23, 2012 · 1 comment
Milestone

Comments

@jstrimpel
Copy link

We are using https://github.com/jzaefferer/grunt-css to add csslint to our build process. According to https://github.com/stubbornella/csslint/wiki/Beware-of-box-model-size:

"If the box-sizing property is specified, then the rule does not emit any warnings for the above conditions as it assumes you know what you're doing."

However, the CSS below is still throwing error box-model rule errors.

.pa-header {
box-sizing: border-box;
background: url('../images/bg_hdr_grad.gif') repeat-x;
height: 33px;
width: 982px;
border-top: #b2c5d3 1px solid;
border-right: #b2c5d3 1px solid;
border-left: #b2c5d3 1px solid;
}

[L6:C3]

Using height with border-top can sometimes make elements larger than you expect. Don't use width or height when using padding or border. (box-model)
[L7:C3]
Using width with border-right can sometimes make elements larger than you expect. Don't use width or height when using padding or border. (box-model)
[L8:C3]
Using width with border-left can sometimes make elements larger than you expect. Don't use width or height when using padding or border. (box-model)
Task "csslint:base_theme" failed. Use --force to continue.

Aborted due to warnings.

@nzakas
Copy link
Contributor

nzakas commented Aug 23, 2012

That definitely seems broken, I'll look into it.

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

No branches or pull requests

2 participants