Conversation
klboke
left a comment
There was a problem hiding this comment.
发现 3 个会直接影响现有预览能力的 blocker:
-
cadviewer相关静态 HTML 被删除后,空间对象面板会直接 404。这个 PR 删除了server/src/main/resources/static/cadviewer/app/cv/cv-pro/space/html/nav_admin_SpaceObjects_B_1_05.html、nav_admin_SpaceObjects_C_1_06.html、nav_user_SpaceObjects_A_3_3_01.html等文件,但server/src/main/resources/static/cadviewer/app/cv/cv-pro/cadviewer.min.js:31281、:31613、:48343仍然在运行时通过.load(...)请求这些路径。结果不是“少几个 demo 文件”,而是 CADViewer 的空间对象/admin 面板会在点击时直接失败。 -
xmind预览的新srcdoc页面引用了仓库里不存在的 jQuery 路径。server/src/main/resources/static/xmind/xmind.js:21现在写的是js/jquery.min.js,但仓库里实际存在的是server/src/main/resources/static/js/jquery-3.6.1.min.js,并没有jquery.min.js。与此同时这个 PR 还删除了原本可工作的server/src/main/resources/static/xmind/index.html,所以 XMind 预览会改走这条新路径并在 iframe 内拿到 404,导致预览初始化失败。 -
OFD 页码跳转和移动端按宽度适配回归了。
server/src/main/resources/web/ofd.ftl:141-161不再通过?page=/?scale=width进入cnofd-view.js既有流程,而是只保存了一个本地变量pageNum后手动调用kkPage()。但server/src/main/resources/static/ofd/js/cnofd-view.js:8-12、:19-20、:32依赖的是window.page和window.location.search里的page/scale参数;现在pageNum没有写回page,所以指定页跳转不会生效,移动端原来的scale=width也丢了。这个改动会把既有的深链和手机预览体验一起打坏。
这三个问题都属于发布前需要先修掉的回归。
No description provided.