-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Fix for issue 4313 selector list not expanding #4314
base: master
Are you sure you want to change the base?
Conversation
* Add support for CSS container query scroll-state.
* Fix an issue where selector with comma would not expand correctly.
* Make fix for issue less#4313 selector list not expanding correctly more flexible.
* Add more tests for fix for issue less#4313.
* Cleanup merge issues for fix for issue less#4313.
Looks like CI may have an issue. Tests pass locally for me.
|
I re-ran CI but ran into issues. Branch passes all tests locally for me. I wouldn't expect files changed to impact CI (two test files, one source file). Would you be able to take a look? @iChenLei |
* Revise CI for missing browser issue.
* Trying to resolve CI issues with Playwright.
* Trying to resolve CI issues with Playwright.
* Trying to resolve CI issues with Playwright.
* Fix CI by pinning CI version to resolve incorrect browser binary downloads.
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. |
* Improve CI stability by only grabbing patch updates. This may when npm install runs we will not download a version of Playwright which will be incompatible with the browser binaries installed by CI. * Add CI code comments about the Playwright issue that we ran into.
* Remove package.json changes per master branch updates.
* Cleanup CI changes per master branch updates to CI.
PR was updated and is now more narrow in scope. Let me know if you have an opportunity to review. @matthew-dean |
What:
PR contains a fix for issue #4313 selector list not expanding with & symbol.
Why:
In Less.js version 4.2.2, this Less:
produces the incorrect CSS:
the & symbol should now be properly expanded with the fix.
Checklist: