Skip to content
This repository has been archived by the owner. It is now read-only.

Disallow 3 term shorthand in Sass #64

Closed
wants to merge 1 commit into from
Closed

Disallow 3 term shorthand in Sass #64

wants to merge 1 commit into from

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Jan 11, 2017

The 3 term shorthand, eg: padding: 5px 0 10px is confusing.

  • Re-enable the check (previously disabled in dcdd0a9)
  • Allow 4 value shorthands (as per #54)

cc @robinwhittleton

The 3 term shorthand, eg: `padding: 5px 0 10px` is confusing.

* Re-enable the check (previously disabled in dcdd0a9)
* Allow 4 value shorthands
@fofr fofr closed this Jan 24, 2017
@fofr
Copy link
Contributor Author

@fofr fofr commented Jan 24, 2017

I'll re-open when we see this has caused a problem.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.