Skip to content

research(ui): 1049 属性保存/恢复的三组交叉验证 (#123/#126) - #184

Merged
JavaZeroo merged 1 commit into
mainfrom
research/1049-attr-confirm
Oct 29, 2026
Merged

JavaZeroo merged 1 commit into
mainfrom
research/1049-attr-confirm

Conversation

@JavaZeroo

Copy link
Copy Markdown
Owner

内容

补齐第三组反向用例,把"进入保存 / 退出恢复"做成三组交叉验证:

组 进入时属性 备用屏动作 退出后(不重设)
1 默认 设为 32;45 默认 (#1C1C1C/#E3E3DD)
2 31;44 不变 31;44 (#3465A4/#C4265E)
3 32;45 reset 成默认 32;45 (#8C6BC8/#86B42B)

第 3 组三阶段像素:进入前 #8C6BC8/#86B42B → 备用屏 reset 后 #1C1C1C/#E3E3DD → 退出后回到 #8C6BC8/#86B42B。

三组一致:1049h 保存进入时刻属性,1049l 恢复它;备用屏内任何 SGR 变更(设置或 reset)都不影响主屏退出后的样式。

边界

Refs #123 #126

…126

Three crossed cases: entry default plus alternate change exits default, entry 31;44 exits 31;44, and entry 32;45 with an alternate-side reset exits 32;45. Alternate-screen changes never leak back.
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ⚠️ Failed 2026-10-29T13:34:51.290574Z
We couldn’t complete this review. Comment @codex review to try again.
594a762 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@JavaZeroo
JavaZeroo merged commit 4a81678 into main Oct 29, 2026
5 of 6 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