Skip to content

Fix issue with :not() selector in nested replacement (issue #1680)#4375

Closed
Keerthanakoduri wants to merge 1 commit intoless:masterfrom
Keerthanakoduri:extend-nexter-selector
Closed

Fix issue with :not() selector in nested replacement (issue #1680)#4375
Keerthanakoduri wants to merge 1 commit intoless:masterfrom
Keerthanakoduri:extend-nexter-selector

Conversation

@Keerthanakoduri
Copy link

Fixes issue #1680 — extended all replacement logic in nested selectors (:not() and similar cases) inside Less.js.

Modified files:

  • selected.js
  • element.js
  • extend-visitor.js

Why:
Previously, the :not() selector handling inside nested replacements caused incorrect behavior.
This PR improves the selector logic to handle nested cases correctly, ensuring accurate CSS output.

Checklist:

  • Documentation – N/A (no docs needed for internal fix)
  • Added/updated unit tests – Verified with existing tests
  • Code complete – Fix implemented and verified locally

This change addresses bug #1680 as described in the issue tracker.
Please review and let me know if any additional test coverage or documentation is required.

@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 21, 2025
@matthew-dean
Copy link
Member

Closing - this PR adds +12,114 lines for a :not() fix, which suggests generated/build files were accidentally included. The fix itself may be valid but needs to be resubmitted cleanly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants