Skip to content
This repository has been archived by the owner on Mar 22, 2019. It is now read-only.

Use shortest unit for zero value #78

Merged
merged 3 commits into from
Jun 22, 2016
Merged

Use shortest unit for zero value #78

merged 3 commits into from
Jun 22, 2016

Conversation

hail2u
Copy link
Owner

@hail2u hail2u commented Jun 22, 2016

Some unit (e.g. ms) cannot be omitted even if its value is 0. However it can be converted to shorter unit (e.g. s).

  • 0ms to 0s
  • 0grad and 0turn to 0deg
  • 0kHz to 0Hz
  • 0dpcm and 0dppx to 0dpi

`0ms` can be written `0s`, same happens on `<resolution>`, `<angle>`,
and `<frequency>`.
@hail2u
Copy link
Owner Author

hail2u commented Jun 22, 2016

But it’s dangerous applying to media queries, disable it.

@hail2u
Copy link
Owner Author

hail2u commented Jun 22, 2016

This updates test for issue #11, because it duplicates property.

@hail2u hail2u merged commit 3aae157 into master Jun 22, 2016
@hail2u hail2u deleted the unit-for-zero branch June 22, 2016 06:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant