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

Tree functions cleanup + CSS Guards default error. #1757

Merged
merged 2 commits into from
Dec 22, 2013
Merged

Tree functions cleanup + CSS Guards default error. #1757

merged 2 commits into from
Dec 22, 2013

Conversation

seven-phases-max
Copy link
Member

These two commits are slightly unrelated, but the second one depends on the first so I decided to combine them in one the PR.

  • Functions.js: Some (minor) refactoring of "externally" defined functions (math, blending, default) - mostly just cleaned up their initialization . No changes in the language / function results.
  • Default function "front-end" reworked for a bit higher-level control from its "back-end" code (e.g. tree.mixin.eval).
  • Added error for default when it's used in CSS guards (default() does not work with CSS guards #1748).

…rnally" defined functions (math, blending, `default`).

* `default` function "front-end" reworked for a bit higher-level control from its "back-end" code (e.g. tree.mixin.eval).
@lukeapage
Copy link
Member

looks good

lukeapage added a commit that referenced this pull request Dec 22, 2013
Tree functions cleanup + CSS Guards `default` error.
@lukeapage lukeapage merged commit 8eea798 into less:master Dec 22, 2013
@seven-phases-max seven-phases-max deleted the tree-functions-cleanup branch December 22, 2013 15:52
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