Skip to content

fix(tooltip): 修复单元格文字过长时 tooltip 显示被截断 close #1028#1034

Merged
xingwanying merged 1 commit intomasterfrom
fix-word
Jan 21, 2022
Merged

fix(tooltip): 修复单元格文字过长时 tooltip 显示被截断 close #1028#1034
xingwanying merged 1 commit intomasterfrom
fix-word

Conversation

@lijinke666
Copy link
Copy Markdown
Member

@lijinke666 lijinke666 commented Jan 21, 2022

👀 PR includes

🎨 Enhance

  • Code style optimization

🐛 Bugfix

  • Solve the issue and close #0

📝 Description

  1. 修复单元格内容过长tooltip显示不全
  2. 简化一些样式
  3. 兼容其他可能存在的过长的场景

image
image image

🖼️ Screenshot

Before After
image image

🔗 Related issue link

🔍 Self Check before Merge

  • Add or update relevant Docs.
  • Add or update relevant Demos.
  • Add or update relevant TypeScript definitions.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 21, 2022

🎊 PR Preview has been successfully built and deployed to https://s2-preview-pr-1034.surge.sh

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 21, 2022

Codecov Report

Merging #1034 (ec3f915) into master (2082c22) will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1034      +/-   ##
==========================================
+ Coverage   71.88%   71.92%   +0.04%     
==========================================
  Files         170      170              
  Lines       12243    12243              
  Branches     2884     2884              
==========================================
+ Hits         8801     8806       +5     
+ Misses       2244     2243       -1     
+ Partials     1198     1194       -4     
Impacted Files Coverage Δ
packages/s2-core/src/facet/base-facet.ts 73.37% <0.00%> (+0.39%) ⬆️
packages/s2-core/src/cell/header-cell.ts 51.94% <0.00%> (+0.48%) ⬆️
...ckages/s2-core/src/interaction/event-controller.ts 81.63% <0.00%> (+0.81%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2082c22...ec3f915. Read the comment docs.

@github-actions
Copy link
Copy Markdown
Contributor

Size Change: -58 B (0%)

Total Size: 189 kB

Filename Size Change
./packages/s2-core/dist/index.min.js 118 kB -115 B (0%)
./packages/s2-core/dist/style.min.css 362 B +6 B (+2%)
./packages/s2-react/dist/index.min.js 67.9 kB +379 B (+1%)
./packages/s2-react/dist/style.min.css 2.77 kB -328 B (-11%) 👏

compressed-size-action

@xingwanying xingwanying merged commit 4e654e7 into master Jan 21, 2022
@xingwanying xingwanying deleted the fix-word branch January 21, 2022 09:12
YardWill added a commit that referenced this pull request Jan 24, 2022
* chore: 🤖 auto sync site s2 lock and changelog [skip ci] (#1004)

* chore(release): 🔖@antv/[email protected]  @antv/[email protected]

* chore(release): 🔖 @antv/[email protected]  @antv/[email protected]

* chore(release): 🤖 @antv/s2-v1.3.0 [skip ci]

* chore(release): 🤖 @antv/s2-react-v1.3.0 [skip ci]

* chore(release): 🔖@antv/[email protected] @antv/[email protected]

* chore(release): 🤖 @antv/s2-v1.4.0 [skip ci]

* chore(release): 🤖 @antv/s2-react-v1.4.0 [skip ci]

* chore(release): 🔖@antv/[email protected] @antv/[email protected]

* chore(release): 🤖 @antv/s2-v1.5.0 [skip ci]

* chore(release): 🤖 @antv/s2-react-v1.5.0 [skip ci]

* chore(release): @antv/[email protected] @antv/[email protected] (#1003)

* chore(release): 🤖 @antv/s2-v1.6.0 [skip ci]

* chore(release): 🤖 @antv/s2-react-v1.6.0 [skip ci]

* chore: 🤖 auto sync site s2 lock and changelog [skip ci]

Co-authored-by: yingying.xwy <[email protected]>
Co-authored-by: semantic-release-bot <[email protected]>
Co-authored-by: xinhui.zxh <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Update base-facet.ts (#1009)

手机端滑动问题修改

* docs: 修复文档404链接和报错demo (#1025)

* chore(ci): 新增 prerelease 推送 & CI 细节调整 (#1027)

* chore(ci): 新增 prerelease 推送 & ci 细节调整

* chore(ci): 去掉 --ignore-scripts

* chore(ci): 去掉 --ignore-scripts

* fix(facet): scroll speed options & pagination totals (#1031)

* fix: scrollSpeedRatio value

* fix: ts err

* fix: revert change

* fix: pagination total

* fix(s2-core): 修复延迟销毁表格时没有移除canvas节点的问题 close #1011 (#1033)

* docs: 增加小册子demo《哲学家的观点》 (#1035)

* fix(tooltip): 修复单元格文字过长时 tooltip 显示被截断 close #1028 (#1034)

* feat: 支持配置前端计算小计/总计 (#921)

* feat: surport caculate totals in pivot data set

* test: add number calculate and totals calculate test

* refactor: change totals type to readonly

* refactor: code style

* refactor: code style and optimize test

* docs: add calcTotals's docs and demos

* fix: get cell data

* test: table facet test's width

* feat: 重构前端计算汇总值

* test: 增加前端计算totals测试

* refactor: 抽离getAggregationAndCalcFuncByQuery到utils

* test: 增加getAggregationAndCalcFuncByQuery的测试

* docs: 小计总计文档更新

* fix(tooltip): 修复明细表列头的 tooltip 内容被错误的格式化 close #998 (#1036)

* fix(tooltip): 修复明细表列头的 tooltip 内容被错误的格式化 close #998

* chore: 还原 meta

* test: 修复测试

* fix(interactive): table move problem (#1013)

* fix: table move problem

* fix: code

* fix: v

* fix: move

* fix: calculateInViewIndexes

* feat: update table facet

* fix: v

* fix: code 整理

* fix: 修复测试用例

Co-authored-by: xinhui.zxh <[email protected]>

* docs: 优化demo《哲学家的观点》 (#1038)

* feat: 完善交叉表分页功能 (#1037)

* chore: s2-react watch script

* fix: maxHeight等于offset数组值时下标返回偏大

* fix(s2react): 交叉表初始化未更新分页total

* fix(facet): 序号列无法渲染

* fix(pivotfacet): 渲染行数返回错误

* fix(facet): 按视窗高度剪裁行头和序号列

* fix: 计算文字位置时应使用viewportHeight视窗高度

* fix(facet): rowIndexHeader可能为null

* fix(facet): 序号列应早于列头添加到group (#1040)

* chore(release): @antv/[email protected] @antv/[email protected] (#1039)

* chore(release): @antv/[email protected] @antv/[email protected] (#1039) (#1041)

This reverts commit 0d538f2.

* chore(release): @antv/[email protected] @antv/[email protected]

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: yingying.xwy <[email protected]>
Co-authored-by: semantic-release-bot <[email protected]>
Co-authored-by: xinhui.zxh <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: gehaiyi <[email protected]>
Co-authored-by: Jinke Li <[email protected]>
Co-authored-by: yangzhanmei <[email protected]>
Co-authored-by: 刘嘉一 <[email protected]>
@lijinke666
Copy link
Copy Markdown
Member Author

🎉 This PR is included in version @antv/s2-v1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lijinke666
Copy link
Copy Markdown
Member Author

🎉 This PR is included in version @antv/s2-react-v1.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lijinke666
Copy link
Copy Markdown
Member Author

🎉 This PR is included in version @antv/s2-v1.7.0-alpha.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lijinke666
Copy link
Copy Markdown
Member Author

🎉 This PR is included in version @antv/s2-react-v1.7.0-alpha.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants