Skip to content

chore: remove dead code, fix misleading API docs - #312

Merged
xidedix merged 5 commits into
v5.7from
chore/dead-code-cleanup
Oct 19, 2026
Merged

xidedix merged 5 commits into
v5.7from
chore/dead-code-cleanup

Conversation

@xidedix

@xidedix xidedix commented Oct 19, 2026

Copy link
Copy Markdown
Member

Dead code and misleading API docs in shared code, no behaviour change.

  • Tooltip, Popover: unreachable hidden branch in addTooltipElement removed; private getUID copies replaced by UIDService (3 no-useless-assignment warnings gone); spec for host aria-describedby matching generated id.
  • SidebarNav: fully commented-out sidebar-nav-group.component.ts and commented exports removed.
  • ThemeDirective: dark/colorScheme descriptions no longer name the other input; host components expose only one of them. Docs API tables follow on regeneration.
  • WidgetStatF: exportAs: 'cWidgetStatF, cWidgetStatB', #w="cWidgetStatF" now resolves, old name kept.
  • WidgetStatC: unreachable default progress template removed.
  • typescript-eslint 8.72.0.

Verified: lint, full suite, lib build; tooltip, popover, widgets and sidebar docs pages render and behave the same against the local build.

…ranch

Private getUID copies replaced (3 no-useless-assignment warnings gone); addTooltipElement only runs while visible. Spec: host aria-describedby matches generated id.
… input

Host components expose only one of them; dark precedence kept for cTheme.
…nreachable progress template

cWidgetStatB kept as alias on c-widget-stat-f.
@xidedix
xidedix merged commit 4b81dc9 into v5.7 Oct 19, 2026
9 checks passed
@xidedix
xidedix deleted the chore/dead-code-cleanup branch October 19, 2026 22:37
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.

1 participant