Skip to content

fix(web): show model shortcut and keep dialogs interactive - #863

Merged
rynfar merged 1 commit into
pylonfrom
upstream/2026-09-26-ui-11593
Sep 27, 2026
Merged

rynfar merged 1 commit into
pylonfrom
upstream/2026-09-26-ui-11593

Conversation

@rynfar

@rynfar rynfar commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

Pylon's model picker tooltip omitted its configured keyboard shortcut, and shared dialog popups inherited desktop titlebar dragging. The tooltip now includes the resolved modelPicker.toggle shortcut when keybindings are available, and dialog popups declare no-drag so their controls remain interactive in desktop titlebar areas.

Source: frozen T3 commit 24b711b7f9ccba03518551991e466e2e4f8ace5a (#11593), partial adoption. The z-30 additions to ModelPickerSidebar and shared ProviderInstanceIcon are omitted: Pylon has no confirmed icon overlay failure, and raising the shared icon across settings, sidebar, usage, and composer surfaces could alter stacking. The server Forgejo and iOS terminal hunks are outside this web UI PR and are being handled separately.

Validation: 129 focused tests passed; web typecheck passed; changed-file lint passed. In an isolated web session, the onboarding dialog's computed -webkit-app-region was none before and no-drag after. The browser rendering is otherwise visually unchanged, as expected for an Electron drag-region fix. Before and after screenshots are attached in the UI evidence comment.

Model: GPT-6 Sol, Pylon harness.

@vercel

vercel Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
pylon-marketing Ignored Ignored Preview Sep 27, 2026 3:36am UTC

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S labels Sep 27, 2026
@rynfar

rynfar commented Sep 27, 2026

Copy link
Copy Markdown
Collaborator Author

UI evidence from isolated Pylon web session. The visual layout is unchanged; the onboarding dialog computed -webkit-app-region changed from none before to no-drag after. This CSS behavior is relevant to Electron titlebar interaction.

Before:

before-dialog

After:

after-dialog

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 14.0 KiB 14.0 KiB +21 B (+0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 7.3 KiB 7.3 KiB +14 B (+0.2%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.7 KiB 6.8 KiB +7 B (+0.1%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB +44 B (+0.1%) 66.4 KiB ✅
Codex Live turn messages 9 10 +1 (+11.1%) 21 ✅
Claude Total thread wire 14.0 KiB 14.0 KiB +16 B (+0.1%) 15.1 KiB ✅
Claude Thread snapshot wire 7.3 KiB 7.2 KiB −9 B (−0.1%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.8 KiB +25 B (+0.4%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.9 KiB 58.9 KiB +44 B (+0.1%) 66.4 KiB ✅
Claude Live turn messages 9 10 +1 (+11.1%) 21 ✅

Baseline: 69f26bc · PR result: 6542a7a · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 115.7 KiB
  • Claude decoded thread snapshot: 116.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@rynfar
rynfar merged commit dd4e28b into pylon Sep 27, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant