Skip to content

fix(chat): rename chat-sidenav-scrim output close → dismiss#485

Merged
blove merged 2 commits into
mainfrom
claude/fix-scrim-output-name
May 20, 2026
Merged

fix(chat): rename chat-sidenav-scrim output close → dismiss#485
blove merged 2 commits into
mainfrom
claude/fix-scrim-output-name

Conversation

@blove
Copy link
Copy Markdown
Contributor

@blove blove commented May 20, 2026

Summary

  • Renames ChatSidenavScrimComponent's close output to dismiss.
  • close is a standard DOM event name, which trips @angular-eslint/no-output-native and was the lone error failing Library — lint / test / build after PR Mobile sidenav scrim + drawer elevation + z-index tokens #477.
  • Updates the only consumer (the chat demo shell template) and the component spec + docstring.

3 files, 8 insertions, 8 deletions.

Test plan

  • npx nx run chat:lint → success (was failing on the close output).
  • CI: full Library lint/test/build to confirm green.

🤖 Generated with Claude Code

`close` is a standard DOM event name, which triggers
@angular-eslint/no-output-native and fails Library lint CI. Renames the
EventEmitter to `dismiss`, updates the only consumer
(examples/chat/angular demo shell) and the component spec + docstring.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment May 20, 2026 3:17pm

Request Review

@blove blove enabled auto-merge (squash) May 20, 2026 15:13
@blove blove merged commit b3763a9 into main May 20, 2026
2 checks passed
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