Repository navigation
Show local Windows administrator shields across tab layouts - #449
Merged
Kuddev merged 7 commits intoOct 6, 2026
Merged
Conversation
Owner
|
已按维护者确认的设计直接调整,并同步最新 main(包含 #496、#497)。当前 head:
格式、架构、平台条件预算、diff 检查已通过。本地 Windows 测试构建以退出码 1 结束,输出未包含编译错误或构建完成记录,正在核对进程状态;新 head 的上游检查也已触发;旧 head 的通过结果不当作新 head 的验收结果。 |
Kuddev
approved these changes
Oct 6, 2026
Kuddev
left a comment
Owner
There was a problem hiding this comment.
复核当前 head db16e4e。按维护者最终确认,侧栏、顶部标签和侧栏收起标题均采用“盾牌 → 程序/AI 图标 → 名称”;盾牌与任务图标共用 TAB_LABEL_ICON_SIZE(15 逻辑像素),使用编译期嵌入的矢量资源,保留悬停说明与无障碍名称。盾牌独立于程序图标,不闪烁、不添加定时器。
权限判定复用已缓存的 Windows token 查询,只标记确认继承本机上下文的终端,排除 SSH、WSL、缺少上下文和非终端页面。逐项核查了 10 个差异文件,未发现无关功能、推广内容、依赖或持久化变更。
验证:当前 head 的 Linux 实际原生套件与十项必需检查通过;本机 Windows 从该 head 构建的两项定向回归均通过,分别验证等尺寸/顺序/减少动效,以及本机/远端上下文边界。首次本地构建异常退出未给出诊断,确认原进程退出后在未改源码的情况下重新构建成功,保留两次日志。未声称人工 UAC 窗口、读屏或所有 DPI 的视觉验收。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Result / 用户结果
Elevated local Windows terminal tabs show a persistent 🛡 indicator in the sidebar, top tabs and the active title when the sidebar is collapsed. The shared badge has an accessible label and tooltip identifying the local Pebrel administrator context. Closes #436.
Design / 设计边界
Reuse the existing cached Windows token query and show the badge only after confirmed elevation. The focused pane must belong to the local host: managed SSH, nested SSH history scopes and WSL contexts are excluded, including mixed split tabs. Settings/document tabs never acquire the badge. A static indicator stays readable with reduced motion and adds no animation timer. Existing tab activity, selection, close and drag controls remain independent.
Evidence / 验证依据
Final head
e601dd0de58a5eac8f28838ce7c5e037d0d025cc: architecture, lint, PR size and all required reports passed. Upstream validation executed Linux; contributor fork run 37101266522 additionally passed all five native platforms on the same head. This is automated native-suite evidence; platform/device UI limits below still apply.Two self-review passes completed. New regressions exercise real terminal-view context classification and render the shared badge beside a title with reduced motion enabled and disabled. Final-head GitHub Actions validation passed; the final-head contributor-fork five-platform native run also passed. No local automated checks were run. Manual elevated Windows visual/screen-reader acceptance is not claimed.
AI implementation/review: GPT-6.1 SOL sub-agents; coordinator model and reasoning efforts unverified.
Required Review / 必须确认
Additional final-head fork native validation: five-platform run. This does not claim manual UAC or privileged-window visual acceptance.