Skip to content

Switch header sizes for ZHA pairing status card#8078

Merged
bramkragten merged 1 commit intohome-assistant:devfrom
abmantis:zha_pairing_state_header
Jan 4, 2021
Merged

Switch header sizes for ZHA pairing status card#8078
bramkragten merged 1 commit intohome-assistant:devfrom
abmantis:zha_pairing_state_header

Conversation

@abmantis
Copy link
Copy Markdown
Member

@abmantis abmantis commented Jan 3, 2021

Proposed change

Since the top title on the ZHA device pairing status card shows the result of the previous step ("Interview Complete", "Configuration Complete", etc), while the second title shows the current step ("Configuring", "Initializing", etc), having the top title bigger would draw attention to it. So a user could see "Configuration Complete" and assume pairing is done.

This change makes the second title bigger than the top one, so that the user focus on the step that is in progress.

Before:
image
(at this stage, the interview is complete, but the pairing is still configuring the device. A user could look at "Interview Complete" and think pairing has finished)

After:
image
image
image

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue or discussion:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

Since the top title shows the result of the previous step ("Interview
Complete", "Configuration Complete", etc), while the second title shows
the current step ("Configuring", "Initializing", etc), having the top
title bigger would draw attention to it. So a user could see
"Configuration Complete" and assume pairing is done.

This change makes the second title bigger than the top one, so that the
user focus on the step that is in progress.
@bramkragten bramkragten merged commit 6fb2068 into home-assistant:dev Jan 4, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jan 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants