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

fix:修复手机端部分区域不能滑动问题 #1009

Merged
merged 1 commit into from Jan 17, 2022

Conversation

gehaiyi
Copy link
Contributor

@gehaiyi gehaiyi commented Jan 17, 2022

手机端滑动问题修改

👀 PR includes

✨ Feature

  • New feature

🎨 Enhance

  • Code style optimization
  • Refactoring
  • Change the UI
  • Improve the performance
  • Type optimization

🐛 Bugfix

  • Solve the issue and close #0

🔧 Chore

  • Test case
  • Docs / demos update
  • CI / workflow
  • Release version
  • Other ()

📝 Description

🖼️ Screenshot

Before After

🔗 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
Contributor

github-actions bot commented Jan 17, 2022

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

@xingwanying
Copy link
Member

image

修改下pr标题呢~

Copy link
Member

@xingwanying xingwanying left a comment

Choose a reason for hiding this comment

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

LG

@codecov
Copy link

codecov bot commented Jan 17, 2022

Codecov Report

Merging #1009 (a6be122) into master (c49310d) will not change coverage.
The diff coverage is 50.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1009   +/-   ##
=======================================
  Coverage   71.88%   71.88%           
=======================================
  Files         170      170           
  Lines       12243    12243           
  Branches     2884     2884           
=======================================
  Hits         8801     8801           
  Misses       2244     2244           
  Partials     1198     1198           
Impacted Files Coverage Δ
packages/s2-core/src/facet/base-facet.ts 72.97% <50.00%> (ø)

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 004a98d...a6be122. Read the comment docs.

@gehaiyi gehaiyi changed the title Update base-facet.ts fix: 修复手机端部分区域不能滑动问题 Jan 17, 2022
@github-actions github-actions bot added the pr(fix) bug fix label Jan 17, 2022
@xingwanying xingwanying changed the title fix: 修复手机端部分区域不能滑动问题 fix(s2-core): 修复手机端部分区域不能滑动问题 Jan 17, 2022
@gehaiyi gehaiyi changed the title fix(s2-core): 修复手机端部分区域不能滑动问题 fix:修复手机端部分区域不能滑动问题 Jan 17, 2022
@github-actions github-actions bot removed the pr(fix) bug fix label Jan 17, 2022
@xingwanying xingwanying merged commit 8b37d47 into antvis:master Jan 17, 2022
@xingwanying
Copy link
Member

close #1009

@lijinke666
Copy link
Member

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

The release is available on:

Your semantic-release bot 📦🚀

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/s2@1.2.0  @antv/s2-react@1.2.0

* chore(release): 🔖 @antv/s2@1.2.0  @antv/s2-react@1.2.0

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

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

* chore(release): 🔖@antv/s2@1.4.0 @antv/s2-react@1.4.0

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

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

* chore(release): 🔖@antv/s2@1.5.0 @antv/s2-react@1.4.1

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

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

* chore(release): @antv/s2@1.6.0 @antv/s2-react@1.6.0 (#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 <yingying.xwy@antgroup.com>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>
Co-authored-by: xinhui.zxh <xinhui.zxh@antgroup.com>
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 <xinhui.zxh@antgroup.com>

* 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/s2@1.7.0 @antv/s2-react@1.7.0 (#1039)

* chore(release): @antv/s2@1.7.0 @antv/s2-react@1.7.0 (#1039) (#1041)

This reverts commit 0d538f2.

* chore(release): @antv/s2@1.7.0 @antv/s2-react@1.6.1

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: yingying.xwy <yingying.xwy@antgroup.com>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>
Co-authored-by: xinhui.zxh <xinhui.zxh@antgroup.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: gehaiyi <1105993686@qq.com>
Co-authored-by: Jinke Li <a1231236677287@163.com>
Co-authored-by: yangzhanmei <15809244435@163.com>
Co-authored-by: 刘嘉一 <lcx.seima@gmail.com>
@lijinke666
Copy link
Member

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

The release is available on:

Your semantic-release bot 📦🚀

wjgogogo pushed a commit that referenced this pull request Oct 25, 2023
手机端滑动问题修改
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.

None yet

3 participants