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

@import should import ONCE (default behavior) #396

Open
akomm opened this issue Mar 20, 2013 · 2 comments
Open

@import should import ONCE (default behavior) #396

akomm opened this issue Mar 20, 2013 · 2 comments

Comments

@akomm
Copy link

akomm commented Mar 20, 2013

@see: less/less.js#212
They came to the conclusion, that @import default behavior should be to import ONCE. Its already implemented that way.

lessphp dont follow this behavior.

I propose to add the following:

// less.inc.php line 112 in \lessc::tryImport()
if(array_key_exists(realpath($realPath), $this->allParsedFiles())) {
    return array(false, '');
}
// less.inc.php line 45 in \lessc property list
protected $allParsedFiles = array();
@saas786
Copy link

saas786 commented Apr 12, 2013

+1

nielsAD added a commit to nielsAD/lessphp that referenced this issue Jun 2, 2013
@nielsAD nielsAD mentioned this issue Jun 3, 2013
@rulatir
Copy link

rulatir commented Dec 8, 2013

What is stalling this pull?

How can the community help make it happen?

Is the main developer not completely convinced that merging this change is the RightThing™ to do?

Is the main developer not completely convinced that this functionality is a crucial, usefulness-defining feature of the LESS language in its current form?

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

No branches or pull requests

3 participants