Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the sticky layout of Attribute View #9625

Closed

Conversation

Zuoqiu-Yingyi
Copy link
Contributor

  • Please commit to the dev branch
  • For contributing new features, please supplement and improve the corresponding user guide documents
  • For bug fixes, please describe the problem and solution via code comments
  • For text improvements (such as typos and wording adjustments), please submit directly

思源笔记-功能示范-40

TESTED

@Zuoqiu-Yingyi Zuoqiu-Yingyi changed the title Feat/attr view sticky Improve the sticky layout of Attribute View Nov 10, 2023
@88250 88250 requested a review from Vanessa219 November 10, 2023 14:01
@Zuoqiu-Yingyi
Copy link
Contributor Author

v2.10.15-dev2 中属性视图尾行在出现横向滚动条时原方案出现错位问题, 此方案能修复该问题

@Zuoqiu-Yingyi
Copy link
Contributor Author

修复调整可换行的列宽时尾行位置错位问题

@Vanessa219
Copy link
Member

Vanessa219 commented Nov 14, 2023

固定列已完整实现,目前冲突较多,先关闭了。

@Vanessa219 Vanessa219 closed this Nov 14, 2023
@Zuoqiu-Yingyi
Copy link
Contributor Author

固定列已完整实现,目前冲突较多,先关闭了。

别关闭啊, 我兼容一下

@Zuoqiu-Yingyi
Copy link
Contributor Author

@Vanessa219 已解决冲突且完成兼容, 并进行了本地测试

@Vanessa219
Copy link
Member

是要提交什么么?

@Zuoqiu-Yingyi
Copy link
Contributor Author

是要提交什么么?

可以重新打开该 PR, 拉取一下该分支编译测试一下, 目前没发现什么问题嘿嘿

@Vanessa219
Copy link
Member

明早要发版了,目前先不动了。

上次看代码有点多,和原有设计也有冲突。最新的是修复了什么问题?

@Zuoqiu-Yingyi
Copy link
Contributor Author

明早要发版了,目前先不动了。

不用急

上次看代码有点多,和原有设计也有冲突。最新的是修复了什么问题?

  • 一劳永逸解决首行与尾行类粘性布局错位的问题
  • 优化了 .av__colsticky 元素的 DOM 结构 (将 .av__firstcol 元素放在 .av__colsticky 元素内部, 避免 .av__colsticky.av__firstcol 元素的出现, 更方便第三方主题更改其样式)
  • 修复了 .b3-chip 元素在竖直方向偏移的问题
  • 优化了单元格中文本换行的方案 (使用 HTML 属性 data-wrap="true"data-wrap="false" 设置, 这样更方便第三方主题更改其样式)

@Vanessa219
Copy link
Member

分成不同的 PR 吧,这样不会因为其他问题导致全部都无法合并。我也比较好 review,否则 merge 了出现问题又需要 revert 了。

这样发布公告也比较清晰,每个 issue 是做了什么。代码追踪也比较好做一点。

@Zuoqiu-Yingyi
Copy link
Contributor Author

分成不同的 PR 吧,这样不会因为其他问题导致全部都无法合并。我也比较好 review,否则 merge 了出现问题又需要 revert 了。

这样发布公告也比较清晰,每个 issue 是做了什么。代码追踪也比较好做一点。

可以, 但是最好都尽快合并, 我没有精力同时维护太多的 PR
这次这么多更改就是因为上上次 PR 用了一个多月合并导致许多小修小补都塞到一起了

@Zuoqiu-Yingyi
Copy link
Contributor Author

  • 解决首行与尾行类粘性布局错位的问题 Improve the sticky layout of Attribute View rows #9655
  • 剩下的修改耦合度较高, 未避免意料外的冲突, 需要等待 Improve the sticky layout of Attribute View rows #9655 合并后再进行测试与 PR
    • 优化 .av__colsticky 元素的 DOM 结构 (将 .av__firstcol 元素放在 .av__colsticky 元素内部, 避免 .av__colsticky.av__firstcol 元素的出现, 更方便第三方主题更改其样式)
    • 修复 .b3-chip 元素在竖直方向偏移的问题
    • 优化单元格中文本换行的方案 (使用 HTML 属性 data-wrap="true"data-wrap="false" 设置, 这样更方便第三方主题更改其样式)

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.

2 participants