-
-
Notifications
You must be signed in to change notification settings - Fork 825
Make app widget available to Samsung Z Flip flex window #5753
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
|
I'm pretty new to android development I don't know why the cell width isn't taken in consideration, on newer version of the flex window you can have multiple widget on the same screen. |
|
Samsung seams to not use at all the target cell and want a fixed size for widgets, I'm not sure if there is a way to detect that you are on the cover screen and use onUpdate to have a fixed size only there |
After looking at Samsung's documentation you linked: maybe it is because the Samsung-specific metadata XML is missing, have you tried adding that? (Looks like one XML could be used for all widgets as it has just one property.) |
see samsung references :
Summary
Add keyguard widget category to all widget, doing so allow homeassistant widgets to be listed in cover screen option on z flip line of products (I own a broken z flip 5 and z flip 7)
Checklist
Screenshots
https://github.com/user-attachments/assets/c2c574a7-68fd-42ed-b1c2-b51cb482009d
Link to pull request in documentation repositories
User Documentation: home-assistant/companion.home-assistant#
Developer Documentation: home-assistant/developers.home-assistant#
Any other notes