lit/lit

[Analyzer] Add a way to add inherit properties to element in manifest instead of superclass

Open

#4,529 opened on Jan 29, 2024

View on GitHubContributor guide
 (1 comment) (0 reactions) (0 assignees)TypeScript (874 forks)batch import
Good First Issue

Repository metrics

Stars
 (16,966 stars)
PR merge metrics
 (Avg merge 8d 10h) (3 merged PRs in 30d)

Description

Should this be an RFC?

  • This is not a substantial change

Which package is this a feature request for?

Analyzer (@lit-labs/analyzer)

Description

Components that inherit other Lit webcomponents don't inherit properties (but just set the superclass field). This breaks the compatibility with external tools. See: https://github.com/break-stuff/cem-tools/issues/69

Alternatives and Workarounds

CEM analyzer works fine

Contributor guide

Research direction
Examine the current analyzer code to understand how superclass properties are inherited and modify the manifest generation to include inherited properties.
Tech stack
typescript
Domain
toolingdeveloper experience
Issue type
Feature
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.
Half day
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
Litcustom element manifest
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50