Skip to content

feat(schema): document [ALL] and [ADD] badges in schema legend (DRC-3466)#1434

Draft
danyelf wants to merge 3 commits into
mainfrom
feature/drc-3466-schema-legend-should-cover-the-whole-and-add-badges
Draft

feat(schema): document [ALL] and [ADD] badges in schema legend (DRC-3466)#1434
danyelf wants to merge 3 commits into
mainfrom
feature/drc-3466-schema-legend-should-cover-the-whole-and-add-badges

Conversation

@danyelf

@danyelf danyelf commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Type

  • Enhancement

Description

Extends the schema-view legend (top of the Columns tab) to document the two newer lineage markers that previously had no in-product reference:

  • [ALL] whole-model — every row in this model can be affected
  • [ADD] additive — adds a column; existing rows unchanged

Both entries reuse the existing chip style and wholeModelTreatmentTokens via pickTitleChip / pickGraphBadge + TreatmentChip — no new colors invented. The two new entries render only when new_cll_experience is enabled (the same flag that controls whether the badges render at all in the lineage graph / NodeView), so the legend never documents markers a user can't see. The existing + added / - removed / ~ changed entries are unchanged.

Closes DRC-3466. Follow-up from DRC-3341.

Linked issues

  • DRC-3466 — Schema legend should cover the [WHOLE] and [ADD] badges

How tested

  • SchemaLegend unit tests (8/8 pass): always-on added/removed/changed entries; whole-model + additive entries present with correct copy & aria-labels when new_cll_experience is on; both hidden when off.
  • biome check clean on changed files.
  • tsc --noEmit exit 0.

User-facing changes

The Columns-tab legend now explains the [ALL] and [ADD] badges (when the new CLL experience is enabled). No behavior change to the badges themselves.

Checklist

  • Tests added/updated and passing
  • DCO sign-off on commits

danyelf and others added 2 commits June 15, 2026 16:26
The schema-view legend listed only added / removed / changed, leaving the
ALL (whole-model) and ADD (additive) badges from the lineage view and panel
header undocumented. Add legend entries for both, reusing the existing
treatment tokens so the chips match the in-product badges, gated on the same
new_cll_experience flag that controls whether the badges render.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Danyel Fisher <danyel@gmail.com>
@danyelf danyelf marked this pull request as draft June 18, 2026 00:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants