rahul37wallst-sudo opened a new pull request, #21704: URL: https://github.com/apache/echarts/pull/21704
## Brief Information This pull request is in the type of: - [x] bug fixing - [ ] new feature - [ ] others ### What does this PR do? Corrects the alignment of rich treemap labels using `insideBottom`, `insideBottomLeft`, or `insideBottomRight`. ### Fixed issues - #17591: Rich treemap labels configured with `insideBottomLeft` were displayed at the top of their cells. ## Details ### Before: What was the problem? Treemap labels were always assigned the full available cell height. ZRender interprets an explicit height as the height of the entire rich-text block, so bottom-aligned rich labels kept their text tokens at the top of that block and appeared at the top of the treemap cell. Plain-text labels were not affected in the same way. ### After: How does it behave after the fixing? Rich labels configured with `insideBottom`, `insideBottomLeft`, or `insideBottomRight` now use their natural content height and align correctly at the bottom of the treemap cell. The change is intentionally limited to these three positions. Plain labels and rich labels at top or center positions retain their existing height constraints, minimizing unrelated visual changes. Unit tests cover multiline labels with mixed rich and plain content, all three bottom positions, and compatibility controls for unaffected positions. A focused visual test case was also added. ## Document Info One of the following should be checked. - [x] This PR doesn't relate to document changes - [ ] The document should be updated later - [ ] The document changes have been made in apache/echarts-doc#xxx ## Misc ### Security Checking - [ ] This PR uses security-sensitive Web APIs. ### ZRender Changes - [ ] This PR depends on ZRender changes (ecomfe/zrender#xxx). ### Related test cases or examples to use the new APIs - `test/treemap-label-position-rich.html` - `test/ut/spec/series/treemap.test.ts` ### Merging options - [ ] Please squash the commits into a single one when merging. ### Other information Validated with: - Focused Jest tests: 5 tests passed - `npm run lint` - `npm run checktype` - Local bundle build - SVG renderer validation confirming zero position difference between equivalent plain and rich bottom-aligned labels -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
