linkedin / linkedin/css-blocks

Better support progressive enhancement in Conflict Validator and Conflict Resolver

Open
#69 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_r167104549

After #62 lands we should not have problems with uncaught conflicts. However, there is room for improvement with how css-blocks handles progressive enhancement and vendor prefixed property resolutions. Ex:

```css
block-a.foo {
-webkit-transform: translateY(50%);
transform: translateY(50%);
}

block-b.foo {
-webkit-transform: translateX(50%);
transform: translateX(50%);
-webkit-transform: resolve('block-a.foo;'); /* Do we need this resolution if we're PA aware? */
transform: resolve('block-a.foo;');
}
```

Contributor guide

Open the contributing guide

Research direction

Start by reading the discussion on PR #62 and the progressive-enhancement example in this issue. Define the expected handling of vendor-prefixed properties and whether matching resolve declarations remain necessary; done means the Conflict Validator and Conflict Resolver handle these cases without uncaught conflicts.

Written by the indexing model from the issue text.

Assessment

Tech stack
css, typescript
Domain
frontend
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.