Skip to content

fix(updates): size update center to active content - #14765

Open
ThugKd wants to merge 3 commits into
t8y2:mainfrom
ThugKd:feature/stable-update-center-height
Open

ThugKd wants to merge 3 commits into
t8y2:mainfrom
ThugKd:feature/stable-update-center-height

Conversation

@ThugKd

@ThugKd ThugKd commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

更新中心切换 DBX、驱动、MCP 等标签时,弹窗高度随当前内容变化。高度上限为 720 像素或窗口高度减 48 像素,取较小值。超出时正文内部滚动,标题、标签和底部操作保持可见。切换标签时从正文顶部开始显示。

Closes #14762.

验证

  • 更新中心及相关的 6 个现有测试文件、177 项测试通过。类型检查、生产前端打包、格式、lint 和生成类型检查通过。
  • 在隔离 Chrome 中用模拟数据切换 DBX、驱动、MCP、JDBC、插件和更新日志。测试窗口为 1280×1000、480×640 和 480×420。单条 MCP 更新的弹窗高度为 242 像素,长更新日志分别达到 720、592 和 372 像素的上限,只有正文滚动。
  • 在相同三种窗口尺寸下预览生产包,验证 DBX 与更新日志切换。在 480×420 中两页都需要滚动,切回 DBX 后正文从顶部显示。验证过程中没有执行更新。

@eryajf eryajf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

已审查 exact HEAD 7b319827890d055c01e1e8fd2e0984ce4de768a0。

更新中心改为固定的视口约束高度,内容区继续独立滚动,符合 Issue #14762 的目标。相关组件测试、旧 WebView 回退测试、typecheck、lint、格式检查及当前 exact-head CI 均已通过。仅建议后续补充 480px 窄视口的独立布局证据,不阻塞本次合并。

@t8y2

t8y2 commented Nov 11, 2026

Copy link
Copy Markdown
Owner

感觉固定住之后下面一大片空白有点奇怪

@eryajf

eryajf commented Nov 12, 2026

Copy link
Copy Markdown
Collaborator

确实,做成自适应效果(且有最大上限)是不是更好

@ThugKd ThugKd changed the title fix(updates): keep update center height stable fix(updates): size update center to active content Nov 12, 2026
@6LWa6ZKx

Copy link
Copy Markdown

顶部固定,内容区往下自适应的高度怎么样?这样鼠标点击tab不会高高低低、跑来跑去

@6LWa6ZKx

Copy link
Copy Markdown
2026-11-13_140743_103.mp4

例如图中的情况,鼠标按钮点击Tab切换不统一

zipg
zipg previously requested changes Nov 13, 2026

@zipg zipg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

自适应高度符合此前关于空白过多的反馈,但当前实现还有交互问题,需要调整后复测。请在后续更新中补充调整前后截图,以及短内容/长更新日志连续切换的录屏,至少覆盖桌面和窄窗口。


<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">

@zipg zipg Nov 13, 2026 •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[P2] 自适应高度时需要固定标签栏的位置。这里从 flex-1 改成 flex-auto 后,不同标签的内容会改变整个 DialogContent 的高度,但通用 DialogContent 的 positioner 仍使用 grid place-items-center 居中,UpdateDialog 也没有覆盖 portalClass。切换短 MCP 内容与长更新日志时,标题和标签栏会跟着上下移动,连续点击时鼠标原位置不再是同一个标签。PR 最新评论也反馈了同一现象。请在保留最大高度和正文滚动的同时,让切换标签时顶部位置稳定,并在桌面及窄窗口中验证。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

已在 e2caba279 固定更新中心的顶部位置。短内容仍随内容收起,长内容达到高度上限后只滚动正文。1280×900、480×640、480×420 三种窗口反复切换时,标签栏位移均为 0 像素。146 项相关测试、类型检查、生产前端打包、格式和 lint 均通过。麻烦复审。

每张对照图上排为修复前,下排为修复后,左侧为短 MCP 内容,右侧为长更新日志。

@zipg
zipg dismissed their stale review November 13, 2026 10:15

作者在 e2caba2 固定更新中心顶部。已独立在桌面与两种窄窗口完成真实组件布局测量、滚动复位、86 项相关测试及完整 Vue 类型检查,原定位问题已解决。CI 尚在排队,另行提交复审通过意见。

@zipg zipg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

复审 exact HEAD e2caba279f930227c61a153486a4d8f4d67a6eb6 通过,上次关于切换标签时顶部位置移动的问题已解决。

本次定位覆盖仅作用于更新中心:按窗口高度确定固定顶部,保留自适应高度、最大高度和正文内部滚动,没有修改通用对话框定位或新增交互入口。已核对补充的桌面/窄窗口修复前后对照图。

独立验证:6 个相关测试文件、86 项测试通过,完整 Vue 类型检查通过。在隔离 Chrome 中加载真实 UpdateDialog 与现有样式,使用模拟更新数据,分别在 1280×900、480×640、480×420 各做 3 轮短 MCP/展开的长更新日志切换,标签栏垂直位移均为 0;长日志高度上限分别为 720/592/372px,正文可独立滚动,切回日志后滚动位置归零,未发现页面横向溢出。此验证不包含实际下载/安装更新。

当前 HEAD 的 CI 仍在排队;代码及布局复审通过,待该提交 CI 通过后再合并。

This branch has not been deployed

No deployments
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.

[Bug] 更新中心切换分类时弹窗高度变化导致误触关闭

6 participants