Skip to content

[WIP]Feat(bar chart) make bar gap in group percentage based on actual bar width #3950 - #3962

Closed
xuefei1313 wants to merge 2 commits into
developfrom
feat(bar-chart)-make-barGapInGroup-percentage-based-on-actual-barWidth-#3950
Closed

xuefei1313 wants to merge 2 commits into
developfrom
feat(bar-chart)-make-barGapInGroup-percentage-based-on-actual-barWidth-#3950

Conversation

@xuefei1313

Copy link
Copy Markdown
Contributor

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Release
  • Site / documentation update
  • Demo update
  • Workflow
  • Other (about what?)

🔗 Related issue link

🔗 Related PR link

🐞 Bugserver case id

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@xuefei1313
xuefei1313 changed the base branch from develop to dev/2.0.0 May 15, 2025 07:48
@xuefei1313
xuefei1313 changed the base branch from dev/2.0.0 to develop May 15, 2025 07:48
@xuefei1313 xuefei1313 changed the title Feat(bar chart) make bar gap in group percentage based on actual bar width #3950 [WIP]Feat(bar chart) make bar gap in group percentage based on actual bar width #3950 May 15, 2025
@xile611

xile611 commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

当前方案会改变 barGapInGroup 百分比值的既有 API 语义:计算基准从 bandWidth 改为实际柱宽,导致已有配置的布局发生变化。暂缓合入并关闭此 PR;#3660 的需求保留,后续明确兼容方案后再推进。

@xile611 xile611 closed this Sep 15, 2026
@Issues-translate-bot

Copy link
Copy Markdown

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿


The current solution will change the existing API semantics of the barGapInGroup percentage value: the calculation basis is changed from bandWidth to the actual column width, resulting in changes to the existing configured layout. The merge of this PR is suspended and closed; the requirements of #3660 are retained and will be advanced after the compatibility plan is clarified later.

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.

3 participants