Skip to content

v0.0.25

v0.0.25: fix(chat): a11y + UX polish caught by live browser smoke (0.0.25) (#198)
@blove blove tagged this 05 May 21:46 
* fix(chat): a11y + UX polish caught by live browser smoke (0.0.25)

Three fixes from continued live Chrome smoke testing of @ngaf/chat 0.0.24:

1. **Textarea auto-resize**: rows="1" with no auto-grow logic meant
   multi-line input via Shift+Enter was hidden behind the fixed 24px
   height (scrollHeight grew to 72px+ while clientHeight stayed at 24px).
   Added an effect() that resizes el.style.height to scrollHeight on
   every messageText change, capped at 200px (~8 lines) before
   switching to overflow scrolling.

2. **Escape closes model picker from trigger**: ChatSelectComponent
   handled Escape only on keydown inside the menu (focused option).
   When user clicked the trigger to open and pressed Escape without
   arrowing into the menu, the keypress hit the trigger which only
   handled Enter/Space/ArrowDown — Escape was ignored and the menu
   stayed open until click-outside. Added Escape branch to
   onTriggerKeydown.

3. **aria-pressed on rating toggle buttons**: thumbs-up/down rendered
   visual is-active class but no aria-pressed attribute. Screen
   readers couldn't communicate toggle state. Added [attr.aria-pressed]
   bound to the rating signal.

Synchronized version bump to 0.0.25 across all 16 @ngaf libs.

* fix(chat): pass [message] to assistant <chat-message> for citations panel

Live Chrome smoke against published 0.0.24 caught: chat-citations is
template-conditioned on @if (message()?.role === 'assistant' && message()),
but the chat composition did not pass [message]="message" to
<chat-message>. So message() was always undefined inside chat-message,
the @if branch was falsy, and <chat-citations> never rendered — even when
markdown citation defs existed in the message content.

This was the missing piece of the 0.0.24 sources panel fix. Three of the
four fixes worked end-to-end (table rows, citation no-url span, task-list
checkbox layout); the panel itself silently never appeared.

Fix: add [message]="message" to the assistant <chat-message> in
chat.component.ts. The user/system variants don't need it.

* fix(chat): clipboard fallback on writeText rejection

Live Chrome smoke caught: when navigator.clipboard.writeText rejects
(permissions, non-secure context, document-not-focused), the catch
silently swallowed the error. The textarea+execCommand legacy path was
only used when clipboard API was missing — not when it failed.

Refactored: try Async Clipboard API first; on rejection, fall through
to the textarea legacy path. Only set copied state when at least one
path succeeded.

* fix(chat): wide-table overflow scroll + broken-image alt fallback

Live Chrome smoke caught:

1. Wide tables (8+ columns or long content) had no overflow handling.
   chat-md-table used 'display: contents' so the inner <table> dictated
   width, with parent at viewport width — wider tables overflowed
   horizontally with no scroll wrapper. Fixed: chat-md-table now uses
   'display: block; overflow-x: auto; max-width: 100%' so wide tables
   scroll within their message bubble. Row/cell elements stay
   layout-transparent so browser table layout still works.

2. Broken images (404 / invalid URL) showed only the browser's default
   broken-image icon with no readable alt text. Fixed: <img> now binds
   (error) to a 'failed' signal that swaps to a styled fallback pill
   showing the alt text and a placeholder icon. Pill uses the same
   surface-alt background as code blocks for consistency.
Assets 2
Loading