apache / apache/echarts

[Bug] Missing funnel chart improvements from #17461

Open
#21,302 2 comments 1 reaction 0 assignees View on GitHub
bug en pending
Dominant language
TypeScript
Stars
67.3k
Forks
19.8k
Avg merge
11d 14h
Merged PRs (30d)
8

Description

### Version

6.0.0

### Link to Minimal Reproduction

https://echarts.apache.org/examples/en/editor.html?c=funnel&code=PYBwLglsB2AEC8sDeAoWtJgDYFMBcya6GOAHmAQOQBiArtNDlpUQL4A0RYwwWkIBVMQwAnCAHNxOEVQhgcAW0qdhAM2AiFAQzDyZsSki2tYAHgBGIgPQA-JOZOCAxqwCkLdBy48s54KUEidFUcHVoRfEJhdAATHS0ANQgcAHdBWAitGIB5aCwATwJVLSwAZxxYL2iMnFLuCMEq6NKtADccAEFSgEltKUagyrYVWFwpaBjA4TiwLQIAbUoAZQALYBTlAwBhLAgnAGtNyiTSuSPu6ABHWggRfKPskRjpSgBdYaJysVqFwaFo6BaBSRGj0RjMEbCMD5EAg1RgpjKQboXCqCgGACMAAZ3JDiNwBLAAGxYvHoPy6YAKAgksmwFIQGJgFZUAAcOKR1QUEGgBFJyNg2gCsGx_K5PKWEAAXiCOXShZKZVRRbiBaUNOjKNAYDhOdFxFpCQAmOlYLTmJhTaqwUprNKiWg4OnoEDAU6QGCyaCnZ4eaJNYRmi1YAAyPMi_2quGg4mZBGxztG4aW0NwVut9MZcZFifQ0NhVHVuxifuqrAFAeIckUKfyaai1r8T2kW14GioAGJVN29dUm88RAB1LMskUVumKEArLSnUrp6JBy0NjPqaBgRWRI1YgWeccCmL5QHcpwACRwEhW6LAIkddJmc1g8x3yFgrRKjpppNgR5BJzOlUTJBX3fSIABYvx_KgLmuW57gA58gLfLAP1gLd2G_IEQUeAdKHgjNEJAgh2XQyDtl2A5cMrYQCOQyJRRIzCqFWdZKIFd5hHLdB3lYABuIA

### Steps to Reproduce

Open the link above or open a funnel example from the documentation
Configure the dynamicHeight option to true (it's actually missing, but it should be available)
Configure the sort option to 'none'

### Current Behavior

The funnel is drawn using the standard view

Image

### Expected Behavior

The funnel representation should use the option to draw the funnel with a different layout (by using proportional height with a V shape, instead of proportional width with a fuzzy shape)

Image

### Environment

```markdown
- OS:
- Browser:
- Framework:
```

### Any additional comments?

The MR https://github.com/apache/echarts/pull/17461/ looks like it is merged and the related issues are closed but it's actually missing from the actual released versions

I'm currently working on a product, which used to represent funnel charts with dynamic height (using a different charting library). As the requirement is to have the same representation, I could only get this by applying a patch locally based on the above mentioned pull request. It would be great if these options would be part of the standard library and it looks like (based on the PR comments) that this was meant to be the case.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.