Skip to content

Preserving importance for everything - #130

Merged
tj merged 4 commits into
stylus:masterfrom
kizu:preserve-importance
Aug 28, 2012
Merged

Preserving importance for everything#130
tj merged 4 commits into
stylus:masterfrom
kizu:preserve-importance

Conversation

@kizu

@kizu kizu commented Aug 24, 2012

Copy link
Copy Markdown
Member

This is a somewhat big thing!

  1. This PR adds fixes some existing problems with importance preserving — the display lacked it, there was added support for display: xxx !important, display: box !important included #72, but it wasn't merged and I tried to give it a try myself. Also, it fixes (adds?) importance preserving to the border and border-radius helpers, so you could override them if you'd want.
  2. This PR adds a looot of generic tests on different CSS properties — so if someone would add something to some property like it was with display, this thing won't break. Sometimes this importance thing can be really critical — you could have some global modifier class with some overrided important styles, and it would be a bummer if they won't override something after nib upgrade. So, I added some test to ensure the better future :)
  3. I made some minor things: removed the legacy support for all the extra border-radiuses — the generic border-radius already was webkit- and official-only. Also, fixed tests that were broken after merging Recent change broke my app =( #125.

Yay for the tests!

tj added a commit that referenced this pull request Aug 28, 2012
Preserving importance for everything
@tj
tj merged commit fa03365 into stylus:master Aug 28, 2012
@tj

tj commented Aug 28, 2012

Copy link
Copy Markdown
Collaborator

awesome! thanks

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