google / jetpack-camera-app

Apache License 2.0
152 stars 29 forks source link

ContentDescription should be null when no need a ContentDescription. #279

Closed Jaehwa-Noh closed 2 weeks ago

Jaehwa-Noh commented 4 weeks ago

https://github.com/google/jetpack-camera-app/blob/21fdaf120d14abfee3e884f6ec90f120c5ddea14/feature/preview/src/main/java/com/google/jetpackcamera/feature/preview/ui/PreviewScreenComponents.kt#L452-L461

I know else condition not meet in current condition, but contentDescription should be null when it is no need. And then stringResource(id = 0) is not null, just a dummy string.