copy: Corrected touch target boundaries for various chart types#794

Open

hannesa2 wants to merge 2 commits into

stacked-bar-highlighter

hannesa2 commented Jun 19, 2026 •

edited

Loading

Collaborator

This is a copy of #791 on upstream.
This enables screenshot compare mechanism @venkatbhoogarbh

hannesa2 force-pushed the stacked-bar-highlighter branch 4 times, most recently from b3119e5 to e2036c3 Compare

June 19, 2026 14:23

hannesa2 commented Jun 19, 2026

Collaborator Author

Screenshot differs: emulatorApi=9 with 11 screenshot(s)

setpoint (old) | diff | actual screenshot (new)

StartTest_smokeTestStart-15-BarChartPositiveNegative-Negative-click.png

StartTest_smokeTestStart-15-BarChartPositiveNegative-Negative-click2020.png

StartTest_smokeTestStart-25-CombinedChartActivity-CombinedChart-click2020.png

StartTest_smokeTestStart-31-ListViewMultiChartActivity-Multiple-click2020.png

StartTest_smokeTestStart-6-LineChartColoredActivity-Colorful-click2020.png

StartTest_smokeTestStart-7-PerformanceLineChart-Performance-click.png

StartTest_smokeTestStart-7-PerformanceLineChart-Performance-click2020.png

StartTest_smokeTestStart-7-PerformanceLineChart-Performance-click7070.png

StartTest_smokeTestStart-8-FilledLineActivity-Filled-click.png

StartTest_smokeTestStart-8-FilledLineActivity-Filled-click2020.png

StartTest_smokeTestStart-8-FilledLineActivity-Filled-click7070.png

venkatbhoogarbh added 2 commits

June 19, 2026 17:39


          Fix: Corrected touch target boundaries and stack selection logic for …

a013471

…Bar charts, Pie charts, Stack charts, and many more


          Replaced forEach with for-in loop to support API 23

253cb01

hannesa2 force-pushed the stacked-bar-highlighter branch from c031559 to 253cb01 Compare

June 19, 2026 15:40

hannesa2 mentioned this pull request

Corrected touch target boundaries for various chart types #791

Open

hannesa2 commented Jun 19, 2026

Collaborator Author

As you @venkatbhoogarbh can see, the click changes here something. Is this a desired behavior ?

hannesa2 changed the title ~~Stacked bar highlighter~~ copy: Corrected touch target boundaries for various chart types

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet