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.

Sign up for GitHub

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

Jump to bottom

Sasuke #4225

Closed
wants to merge 3 commits into from
Closed

Sasuke #4225

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e248da4
phantom error
amanjr09-glitch Aug 31, 2023
37f5b45
phantom js removed
amanjr09-glitch Aug 31, 2023
6d1d914
remove
iChenLei Sep 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion CHANGELOG.md
View file
Open in desktop
Expand Up @@ -523,7 +523,7 @@
- [#804](https://github.com/less/less.js/pull/804) Fix for unnamed parameters test fail & failing test for import-once (@jreading)
- [#796](https://github.com/less/less.js/pull/796) fixed issue #795 (@comfuture)
- [#268](https://github.com/less/less.js/pull/268) Implemented named arguments (@jamesfoster)

### v1.3.0 (2012-03-10)
- [#673](https://github.com/less/less.js/pull/673) mocha found a couple global variable leaks. Here's the fix. (@andrewjstone)
- [#634](https://github.com/less/less.js/pull/634) Add @media bubbling/nesting/merging (@NDMarcel)
Expand Down
16 changes: 8 additions & 8 deletions packages/less/src/less-browser/index.js
View file
Open in desktop
Expand Up @@ -46,14 +46,14 @@
return cloned;
}

// only really needed for phantom
function bind(func, thisArg) {
const curryArgs = Array.prototype.slice.call(arguments, 2);
return function() {
const args = curryArgs.concat(Array.prototype.slice.call(arguments, 0));
return func.apply(thisArg, args);
};
}
// // only really needed for phantom
// function bind(func, thisArg) {
// const curryArgs = Array.prototype.slice.call(arguments, 2);
// return function() {
// const args = curryArgs.concat(Array.prototype.slice.call(arguments, 0));
// return func.apply(thisArg, args);
// };
// }

function loadStyles(modifyVars) {
const styles = document.getElementsByTagName('style');
Expand All @@ -70,7 +70,7 @@
/* jshint loopfunc:true */
// use closure to store current style
less.render(lessText, instanceOptions,
bind((style, e, result) => {

Check failure on line 73 in packages/less/src/less-browser/index.js

View workflow job for this annotation

GitHub Actions / Tests on ubuntu-latest with node18 

'bind' is not defined

Check failure on line 73 in packages/less/src/less-browser/index.js

View workflow job for this annotation

GitHub Actions / Tests on ubuntu-latest with node10 

'bind' is not defined

Check failure on line 73 in packages/less/src/less-browser/index.js

View workflow job for this annotation

GitHub Actions / Tests on ubuntu-latest with node16 

'bind' is not defined

Check failure on line 73 in packages/less/src/less-browser/index.js

View workflow job for this annotation

GitHub Actions / Tests on ubuntu-latest with node19 

'bind' is not defined
if (e) {
errors.add(e, 'inline');
} else {
Expand Down