less/less.js

Container style queries unexpected space

Closed

#4,235 opened on Sep 26, 2023

View on GitHubContributor guide
 (16 comments) (0 reactions) (0 assignees)JavaScript (3,517 forks)batch import
buggood first issuehigh priorityup-for-grabs

Repository metrics

Stars
 (16,977 stars)
PR merge metrics
 (Avg merge 19d 1h) (15 merged PRs in 30d)

Description

To reproduce:

https://lesscss.org/less-preview/#eyJjb2RlIjoiQGNvbnRhaW5lciBzdHlsZSgtLXRoZW1lOiBkYXJrKXtcbiAgLm1haW57XG4gICAgY29sb3I6IzAwMDtcbiAgfVxufVxuXG4iLCJhY3RpdmVWZXJzaW9uIjoiNC4yLjAiLCJtYXRoIjoicGFyZW5zLWRpdmlzaW9uIiwic3RyaWN0VW5pdHMiOmZhbHNlfQ==

@container style(--theme: dark){
  .main{
    color: #000;
  }
}

Current behavior: There's a space between "style" and the bracket.

@container style (--theme: dark)

Expected behavior: No space between "style" and the bracket.

@container style(--theme: dark)

Environment information:

  • less version: 4.2.0

Contributor guide

Research direction
Examine the Less parser's handling of @container directives to find where the extra space is inserted after 'style'.
Tech stack
javascript
Domain
frontend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitNode.js
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75