Skip to content

disabled buttons#3

Merged
argshook merged 2 commits into
mainfrom
disabled-buttons
May 17, 2022
Merged

disabled buttons#3
argshook merged 2 commits into
mainfrom
disabled-buttons

Conversation

@argshook

Copy link
Copy Markdown
Contributor
  • add @styled-icons dev dep
  • make prefix icon grayed out when disabled true

@argshook argshook self-assigned this May 17, 2022
@argshook argshook merged commit ca6bea0 into main May 17, 2022
@argshook argshook deleted the disabled-buttons branch May 17, 2022 15:31
argshook added a commit that referenced this pull request Jun 9, 2022
emrberk added a commit that referenced this pull request Jul 14, 2026
Address review findings on the agent-passive-edit path:

- #8 archived preview: seed a read-only preview from the persisted view
  instead of erroring; NotebookProvider gains a `preview` prop that skips
  live-controller registration and persistence; key <Notebook> on archived
  so restore remounts cleanly. Also fixes the restore-from-search stale
  archived flag this surfaced.
- #6 archive/commit race: commitView re-reads the row inside a Dexie
  transaction and returns committed|archived|deleted, rejecting archived
  and deleted rows atomically with the write.
- #3 reconnect freshness: per-generation guard so an in-flight read that
  spans a reconnect reset no longer seeds freshness for the new connection.
- #7 apply_notebook_state: report state_applied + post_apply_aborted on a
  post-commit abort instead of a generic failure.
- #9 hide-result: bump the freshness seq so a stale agent read is caught.
- #15 agent-change toast a11y: persistent live announcer, return focus to
  the pill, split hover/focus auto-hide pause.
- Surface the active tab's archived state to the agent (workspace + digest).

Tests: freshness generation guard, commitView archived/deleted, and the
apply_notebook_state post-commit abort path.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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