search

takahirom / roborazzi

Make JVM Android integration test visible 🤖📸
https://takahirom.github.io/roborazzi/
Apache License 2.0
651 stars 24 forks source link

Include Disabled semantic property in accessibility explanation #361

Closed lukas-mercari closed 1 month ago

lukas-mercari commented 1 month ago

This PR adds Disabled: "true" to the accessibility explanation when SemanticsProperties.Disabled is set (in Compose code). I also included a test that includes UI elements for all the different semantic properties that are currently being printed.

Background

We started using Dump.AccessibilityExplanation for a11y testing recently, and I noticed when working on Buttons that the output would be the same regardless of them being enabled or disabled. It made me wonder whether the enabled property was set correctly, so I thought it'd be good to reflect this in the test output.

github-actions[bot] commented 1 month ago
Snapshot diff report File name Image
com.github.takahirom
.roborazzi.sample.Co
mposeTest.accessibil
ityExplanation_2_com
pare.png
com.github.takahirom
.roborazzi.sample.Co
mposeTest.accessibil
ityExplanation_compa
re.png