Skip to content

style: optimize spreadSheet's some styles #474

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

Merged
merged 2 commits into from
Oct 25, 2024

Conversation

Cythia828
Copy link
Contributor

optimize some styles

变更类型

请选择以下选项以描述 PR 的类型:

  • Bug 修复(修复现有问题)
  • 新功能(添加了一个功能)
  • 代码优化(性能改进、代码重构)
  • 文档更新
  • 单测新增或修改
  • 其他(请说明):

相关问题

变更内容

  • 子产品内部有一些对SpreadSheet的样式要求,目前根据UI出的图调整了背景色、字体、边框等,粗略将风格调整到与子产品内部的其他表格接近,剩余细节后续有时间再再再调整

详细描述

  • 调整前
    image

  • 调整后
    image

对应 Previewer

optimize some styles
@liuxy0551
Copy link
Collaborator

对应数栈哪个版本,截图选中时的样式,改变 pageSize 时新渲染出来的单元格会不会有样式不统一的问题

@Cythia828
Copy link
Contributor Author

对应数栈哪个版本,截图选中时的样式,改变 pageSize 时新渲染出来的单元格会不会有样式不统一的问题

对应数栈版本6.3
原展示样式:
image
修改后:
image

关于改变 pageSize 时新渲染出来的单元格会不会有样式不统一的问题暂无验证,理论上应该不会,分页新渲染出来的单元格上也是会有具体类名的。但关于边框问题,由于原本就没有上边框,各个使用场景应该是在不同地方加的边框,可能是表格的上边框,也可能是上一个盒子的下边框,所以统一都没有修改这块,需要各个子产品自己修改那条边框线的样式。

目前只针对实时的运行结果验证通过,更多场景待验证。

@LuckyFBB LuckyFBB merged commit 02311ee into DTStack:master Oct 25, 2024
1 check passed
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.

3 participants