Daniel Puckowski

Results 61 comments of Daniel Puckowski

FYI figured out the new CI issue. Had to revise CI file to pin Playwright version to resolve browser binary mismatches. @iChenLei @matthew-dean See files changed in this PR.

PR was updated and is now more narrow in scope. Let me know if you have an opportunity to review. @matthew-dean

Prior comments from https://github.com/less/less.js/pull/4344 : This looks good. I wonder a bit if Less should actually calculate the final value, since the color is static in this case (not from...

You could leverage plugins to help accomplish your goals. ```javascript const less = require('less'); function CountMapKeysPlugin() { this.install = function (less, pluginManager) { const tree = less.tree; pluginManager.addVisitor({ isPreEvalVisitor: true,...

Thank you for reporting the issue. I'll take a look this weekend, see if I can come up with a PR.

PR https://github.com/less/less.js/pull/4344 should resolve this issue. Awaiting peer review before it is merged.

Hello, this issue should be resolved in Less ```4.4.0``` available on npm now at https://www.npmjs.com/package/less/v/4.4.0 This Less in part: ```css background: oklch(from #0000FF calc(l + 0.1) c h); ``` becomes:...

Looks like CI has an issue. I previously resolved this issue by pinning the Playwright version used by CI to match ```package.json```, though that may not be the best solution....

I submitted a few PRs. I expect rebase(s) will be needed in order to merge all of them, so let me know when a rebase is needed.

Thank you for the bug report. I'll see if I can take a look this weekend.