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

Properties merging should work also inside directives #2035 #2121

Merged
merged 6 commits into from
Jul 27, 2014

Commits on Jan 22, 2014

  1. Update README.md

    SomMeri committed Jan 22, 2014
    Configuration menu
    Copy the full SHA
    d717c2b View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2014

  1. Merge https://github.com/less/less.js into rhino

    Conflicts:
    	README.md
    jurcovicovam committed Jul 25, 2014
    Configuration menu
    Copy the full SHA
    0145a54 View commit details
    Browse the repository at this point in the history
  2. Properties merging should work also inside directives less#2035

    Fixes issue less#2035 - property merge inside @font-face. The _mergeRules function is now called also for directives with rules. It used to be called only for rulesets.
    
    I had to turn off jasmine tests for merge.less, because it was replacing all urls by their assumed full paths. For example, the url(something.eot) was changed into url(http://localhost:8081/test/less/something.eot). The result did not matched with expected css and failed.
    
    Note: I'm not sure why values order in source map changed. It does not seem to be caused by my change, it was failing before I made them.
    jurcovicovam committed Jul 25, 2014
    Configuration menu
    Copy the full SHA
    c9c6ddb View commit details
    Browse the repository at this point in the history
  3. Putting readme.md into its original state. Also, travis failed with w…

    …eird
    
    message and I want to see what it will do this time.
    
    Previous travis error:
    >> PhantomJS has crashed. Please read the crash reporting guide at
    >> https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug
    >> report at https://github.com/ariya/phantomjs/issues/new with the crash
    >> dump file attached: /tmp/7685d81d-94f5-1db7-12c9006e-72f80442.dmp 0 [
    >> 'PhantomJS has crashed. Please read the crash reporting guide at
    >> https://github.com/ariya/phantomjs/wiki/Crash-Reporting and file a bug
    >> report at https://github.com/ariya/phantomjs/issues/new with the crash
    >> dump file attached: /tmp/7685d81d-94f5-1db7-12c9006e-72f80442.dmp' ]
    jurcovicovam committed Jul 25, 2014
    Configuration menu
    Copy the full SHA
    f565860 View commit details
    Browse the repository at this point in the history

Commits on Jul 27, 2014

  1. Merge https://github.com/less/less.js into font-face-property-merge-2…

    …-2035
    
    Conflicts:
    	test/sourcemaps/basic.json
    jurcovicovam committed Jul 27, 2014
    Configuration menu
    Copy the full SHA
    8daa61f View commit details
    Browse the repository at this point in the history
  2. Moved @font-face property merge test into urls.less and re-enabled

    merge.less in jasmine tests.
    jurcovicovam committed Jul 27, 2014
    Configuration menu
    Copy the full SHA
    c6cf6a8 View commit details
    Browse the repository at this point in the history