Refactor ButtonWidget to use Jetpack Compose - #6915
Conversation
…n Text if a prior state was loaded
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
ktlint found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
| service = action, | ||
| label = label, | ||
| iconName = selectedIcon.mdiName, | ||
| serviceData = kotlinJsonMapper.encodeToString(MapAnySerializer, actionDataMap), |
| service = action, | ||
| label = label, | ||
| iconName = selectedIcon.mdiName, | ||
| serviceData = kotlinJsonMapper.encodeToString(MapAnySerializer, actionDataMap), |
|
This PR appears to have become a migration to Glance for the widget and migrating the configuration to Compose. Please keep those separate to prevent a huge PR, even if it means a little more work because you have to update the configuration twice (one Compose migration, one changing the logic). |
|
For the configure screen we've just migrated the Entity Widget to the new architecture. I invite you to look at it so you see what we are going to expect. |
| } | ||
|
|
||
| val dbMap: Map<String, Any?> = kotlinJsonMapper.decodeFromString( | ||
| MapAnySerializer, |
| } | ||
|
|
||
| val dbMap: Map<String, Any?> = kotlinJsonMapper.decodeFromString( | ||
| MapAnySerializer, |
Yea no problem, I can make some changes to split this up |
4 participants
Summary
Migration of ButtonWidget from XML to Jetpack Compose & Material 3 for #6305
Checklist
Screenshots
Link to pull request in documentation repositories
User Documentation: home-assistant/companion.home-assistant#
Developer Documentation: home-assistant/developers.home-assistant#
Any other notes