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

Preserving importance for everything #130

Merged
merged 4 commits into from
Aug 28, 2012
Merged

Conversation

kizu
Copy link
Member

@kizu kizu commented Aug 24, 2012

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
Copy link
Collaborator

tj commented Aug 28, 2012

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.

None yet

2 participants