Repository navigation
Conversation
|
感觉固定住之后下面一大片空白有点奇怪 |
|
确实,做成自适应效果(且有最大上限)是不是更好 |
|
顶部固定,内容区往下自适应的高度怎么样?这样鼠标点击tab不会高高低低、跑来跑去 |
2026-11-13_140743_103.mp4例如图中的情况,鼠标按钮点击Tab切换不统一 |
zipg
left a comment
There was a problem hiding this comment.
自适应高度符合此前关于空白过多的反馈,但当前实现还有交互问题,需要调整后复测。请在后续更新中补充调整前后截图,以及短内容/长更新日志连续切换的录屏,至少覆盖桌面和窄窗口。
|
|
||
| <div class="flex min-h-0 flex-1 flex-col"> | ||
| <div v-if="tabs.length" class="px-5 pb-2.5 pt-1.5"> | ||
| <div class="flex min-h-0 flex-auto flex-col"> |
There was a problem hiding this comment.
[P2] 自适应高度时需要固定标签栏的位置。这里从 flex-1 改成 flex-auto 后,不同标签的内容会改变整个 DialogContent 的高度,但通用 DialogContent 的 positioner 仍使用 grid place-items-center 居中,UpdateDialog 也没有覆盖 portalClass。切换短 MCP 内容与长更新日志时,标题和标签栏会跟着上下移动,连续点击时鼠标原位置不再是同一个标签。PR 最新评论也反馈了同一现象。请在保留最大高度和正文滚动的同时,让切换标签时顶部位置稳定,并在桌面及窄窗口中验证。
作者在 e2caba2 固定更新中心顶部。已独立在桌面与两种窄窗口完成真实组件布局测量、滚动复位、86 项相关测试及完整 Vue 类型检查,原定位问题已解决。CI 尚在排队,另行提交复审通过意见。
zipg
left a comment
There was a problem hiding this comment.
复审 exact HEAD e2caba279f930227c61a153486a4d8f4d67a6eb6 通过,上次关于切换标签时顶部位置移动的问题已解决。
本次定位覆盖仅作用于更新中心:按窗口高度确定固定顶部,保留自适应高度、最大高度和正文内部滚动,没有修改通用对话框定位或新增交互入口。已核对补充的桌面/窄窗口修复前后对照图。
独立验证:6 个相关测试文件、86 项测试通过,完整 Vue 类型检查通过。在隔离 Chrome 中加载真实 UpdateDialog 与现有样式,使用模拟更新数据,分别在 1280×900、480×640、480×420 各做 3 轮短 MCP/展开的长更新日志切换,标签栏垂直位移均为 0;长日志高度上限分别为 720/592/372px,正文可独立滚动,切回日志后滚动位置归零,未发现页面横向溢出。此验证不包含实际下载/安装更新。
当前 HEAD 的 CI 仍在排队;代码及布局复审通过,待该提交 CI 通过后再合并。
更新中心切换 DBX、驱动、MCP 等标签时,弹窗高度随当前内容变化。高度上限为 720 像素或窗口高度减 48 像素,取较小值。超出时正文内部滚动,标题、标签和底部操作保持可见。切换标签时从正文顶部开始显示。
Closes #14762.
验证