weixin_39572316
weixin_39572316
2020-12-08 19:18

fix: 去除双轴图默认meta设置

原因: 上个版本和柱线图保持一致,x 轴的 scale 默认设置为 ‘cat’ 类型。 但双轴图的 cat 类型,在某一方有空数据的情况下,会导致渲染错误,(见示例 https://riddle.alibaba-inc.com/riddles/7245c531 粗略看了眼,是 g2 在 syncScale 时,cat类型的时候两个 x 轴的排序没有对上。我再看下是在 g2 上修还是 g2 plot 上修) 这个 pr 先去除掉这个默认设置,随后再 fix cat 分类问题

该提问来源于开源项目:antvis/G2Plot

  • 点赞
  • 写回答
  • 关注问题
  • 收藏
  • 复制链接分享
  • 邀请回答

4条回答

  • weixin_39572316 weixin_39572316 4月前

    默认分类的问题是啥?如果不默认分类,又会有其他问题。

    目前 time 分类存在的问题:1. 存在空数据 2. 某个 data 为空时,坐标问题,如下图

    屏幕快照 2020-11-25 12 14 29

    cat 分类存在的问题:某个数据项为空时,渲染错位(应该是 sync 时排序使得两个 view 的坐标顺序不同了,又因为双轴图只显示了左 view x 坐标,使得看上去错位了)

    提这个 pr,是觉得默认 cat 的问题,比默认 time 的问题更严重,顺车先回滚下,并不能解决上述问题。不过我先关掉好了,这两天直接修 bug 吧

    点赞 评论 复制链接分享
  • weixin_39629969 weixin_39629969 4月前

    Pull Request Test Coverage Report for Build 380841340

    • 0 of 0 changed or added relevant lines in 0 files are covered.
    • No unchanged relevant lines lost coverage.
    • Overall coverage remained the same at 96.59%

    | Totals | Coverage Status | | :-- | --: | | Change from base Build 380228559: | 0.0% | | Covered Lines: | 3948 | | Relevant Lines: | 3986 |

    💛 - Coveralls
    点赞 评论 复制链接分享
  • weixin_39703773 weixin_39703773 4月前

    默认分类的问题是啥?如果不默认分类,又会有其他问题。

    点赞 评论 复制链接分享
  • weixin_39778106 weixin_39778106 4月前

    😭 Deploy PR Preview 609d87249ef0d5799ca42bc43f8b9a5d05baf5c2 failed. Build logs

    🤖 By surge-preview

    点赞 评论 复制链接分享

相关推荐