Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prefer border: 0 over border: none
Although the scss-lint configuration file here enforced `border: none`,
we chatted about it and agreed that we prefer `border: 0`.
  • Loading branch information
lencioni committed Feb 11, 2016
commit 22169e68d956d9e5693ed26ecf42c39378dc7a11
2 changes: 1 addition & 1 deletion .scss-lint.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ linters:

BorderZero:
enabled: true
convention: none
convention: zero

BemDepth:
enabled: true
Expand Down
21 changes: 21 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
- [OOCSS and BEM](#oocss-and-bem)
- [ID Selectors](#id-selectors)
- [JavaScript hooks](#javascript-hooks)
- [Border](#border)
1. [Sass](#sass)
- [Syntax](#syntax)
- [Ordering](#ordering-of-property-declarations)
Expand Down Expand Up @@ -169,6 +170,26 @@ We recommend creating JavaScript-specific classes to bind to, prefixed with `.js
<button class="btn btn-primary js-request-to-book">Request to Book</button>
```

### Border

Use `0` instead of `none` to specify that a style has no border.

**Bad**

```css
.foo {
border: none;
}
```

**Good**

```css
.foo {
border: 0;
}
```

## Sass

### Syntax
Expand Down