Skip to content

Render the license re-login prompt choices vertically - #458

Merged
tiurin merged 3 commits into
mainfrom
devx-1045-render-relogin-prompt-vertically
Aug 13, 2026
Merged

Render the license re-login prompt choices vertically#458
tiurin merged 3 commits into
mainfrom
devx-1045-render-relogin-prompt-vertically

Conversation

@tiurin

@tiurin tiurin commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Motivation

#449 gave the license re-login prompt a second, advertised option (ESC to exit), but left it on the horizontal rendering path. Both labels are flattened into one dimmed hint and appended to a question that already wraps, so the choices read as prose glued to the end of a sentence — and the sentence's own tail (Log) dangles at the wrap point. #457 fixed exactly this for the slow-start recovery prompt; this applies the same treatment to the re-login prompt.

Before / After

Before:

image

After:

image
Aspect Before After
Choices Flattened into one hint: [ENTER to log in again/ESC to exit] One line each: ● [ENTER] Log in again / ○ [ESC] Exit
Prompt text Statement + question, wrapping mid-sentence with Log left dangling Statement only; the actions live in the labels
Selection Key press only ↑/↓ + Enter, or the advertised key directly
Docs

Nothing to document. This changes the layout and wording of one interactive prompt; no command, flag, env var, or documented behavior changes. The decline key advertised in #449 still works the same way, and the manual recovery steps a decline prints are unchanged.

Closes DEVX-1045

@tiurin tiurin added semver: patch docs: skip Pull request does not require documentation changes labels Aug 13, 2026
@tiurin
tiurin marked this pull request as ready for review August 13, 2026 14:55
@tiurin
tiurin requested review from a team and peter-smith-phd as code owners August 13, 2026 14:55
@tiurin
tiurin requested a review from gtsiolis August 13, 2026 14:56

@gtsiolis gtsiolis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Added f343d30 for some minor copy edits.

@tiurin
tiurin enabled auto-merge (squash) August 13, 2026 16:21
@tiurin
tiurin disabled auto-merge August 13, 2026 16:23
tiurin and others added 2 commits August 13, 2026 18:34
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@tiurin
tiurin force-pushed the devx-1045-render-relogin-prompt-vertically branch from f343d30 to 1a4c1de Compare August 13, 2026 16:37
Co-Authored-By: Claude <noreply@anthropic.com>
@tiurin
tiurin enabled auto-merge (squash) August 13, 2026 16:58
@tiurin
tiurin merged commit 6365c1e into main Aug 13, 2026
19 checks passed
@tiurin
tiurin deleted the devx-1045-render-relogin-prompt-vertically branch August 13, 2026 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs: skip Pull request does not require documentation changes semver: patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants