Skip to content
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

Update Wear more details screen to prevent cut off #3899

Merged
merged 2 commits into from
Sep 30, 2023

Conversation

jpelgrom
Copy link
Member

Summary

Updates the more details screen on a watch to prevent content from being cut off. The entity title and toggle were forced onto one line, so with longer entity names and/or larger text sizes it would get cut off.

New design prominently shows the entity icon, which can be tapped to toggle, and the entity title on another line (which can wrap / limit length). Design is very similar to the frontend tile card and default Wear round buttons with icons, so hopefully familiar to users. Colors are not dynamic - on is always yellow, matching the vehicle interface.

Screenshots

Before After, off After, on After, no toggle*
More details screen with text and toggle immediately at the top cut off by the round screen More details screen with a lightbulb icon in a circle that can be tapped, grey for off More details screen with a lightbulb icon in a circle that can be tapped, yellow for on More details screen with a palette icon for a scene, which is not in a circle indicating it cannot be tapped

* and default text size

Link to pull request in Documentation repository

n/a

Any other notes

Copy link
Member

@dshokouhi dshokouhi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great improvement!

…-cut-off

# Conflicts:
#	wear/src/main/java/io/homeassistant/companion/android/home/views/DetailsPanelView.kt
@JBassett JBassett merged commit ae3248c into home-assistant:master Sep 30, 2023
@jpelgrom jpelgrom deleted the wear-more-details-cut-off branch September 30, 2023 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants