linkedin / linkedin/css-blocks

Improve Style Conflict Error Message

Open
#66 1 comment 0 reactions 0 assignees View on GitHub
@css-blocks/core enhancement
Dominant language
TypeScript
Stars
6.3k
Forks
154
PR merge metrics
No merged PRs in 30d

Description

See: https://github.com/css-blocks/css-blocks/pull/62#discussion_r167109504

tl;dr, move template usage data to the error message:

```
The following property conflicts must be resolved for these co-located Styles:

template:
div.class always uses block-a.root (templates/my-template.hbs:10:32)
and conditionally uses block-b.root (templates/my-template.hbs:10:39)

color:
block-a.root (blocks/foo.block.css:3:36)
block-b.root (blocks/b.block.css:1:30)

background-color:
block-a.root (blocks/foo.block.css:3:48)
block-b.root (blocks/b.block.css:1:43)
```

Contributor guide

Open the contributing guide

Research direction

Start with the linked pull-request discussion, then locate the existing style-conflict error construction and any coverage for its output. Trace how template usage data reaches the error, and consider the work complete when the message includes the template and property details shown in the issue example.

Written by the indexing model from the issue text.

Assessment

Tech stack
css, typescript
Domain
frontend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.