Skip to content

Conversation

@PuvvadaBhaskar
Copy link

@PuvvadaBhaskar PuvvadaBhaskar commented Oct 20, 2025

Comments with doubleslash are not stripped before compile #4264

What changes are being made? (What feature/bug is being fixed here?

What: Result 1 Bug Fixed - Line comments now properly stripped
Comprehensive Documentation - 6 guides + 2 verification scripts
All Changes Committed - To branch Bhaskar (commit: 131356e)
@puckowski
#4264

…iables in min()/max() and handling calc() expressions correctly. Operations with CSS variables now work at runtime, and error messages are more precise, showing the exact operation and types. These changes enhance CSS compatibility, reliability, and developer experience.
Fixes:
1. Line comments (//) now properly stripped in permissive parsing contexts
   - Added line comment detection in parser-input.js parseUntil function
   - Handles line comments in custom properties, @supports, and other contexts
   - Test cases added in packages/test-data/less/line-comments/

2. Document extend/variable scope behavior (issue less#3706)
   - Clarified that extend works after variable evaluation
   - Provided 3 workaround patterns: mixins, CSS custom properties, explicit override
   - Test cases added in packages/test-data/less/extend-variable-scope/

Documentation added:
- README-DOCS.md: Navigation index
- QUICK-REFERENCE.md: One-page reference
- EXTEND-VARIABLE-SCOPE-GUIDE.md: Comprehensive guide
- COMPILATION-FLOW-DIAGRAM.md: Visual diagrams
- IMPLEMENTATION-SUMMARY.md: Technical details
- FINAL-SUMMARY.md: Completion summary
- verify-line-comments-fix.js: Verification script
- direct-test.js: Source validation script
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Oct 20, 2025
@PuvvadaBhaskar
Copy link
Author

i request @puckowski to verify my Pull request As fast as possible

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

Labels

size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant