linkedin / linkedin/css-blocks
Improve Style Conflict Error Message
- 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
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