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

Prevent relative time text wrapping in more-info-sun #8051

Merged
merged 1 commit into from
Jan 4, 2021

Conversation

spacegaier
Copy link
Member

@spacegaier spacegaier commented Dec 30, 2020

Breaking change

Proposed change

Follow-up to #7337.
Due to the inline-block that I had to add to get the text-transform to work for the more-info-sun, longer values for the ha-relative-time (e.g. in German, no issue in English since shorter) resulted in two lines being used, despite there being a lot of free space. This PR now ensures that we stay in a single line as intended.

Before:
image

After:
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

Example configuration

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:

@spacegaier spacegaier added this to the 2021.1.0 milestone Dec 31, 2020
@bramkragten bramkragten merged commit 2113ea6 into home-assistant:dev Jan 4, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jan 5, 2021
@spacegaier spacegaier deleted the follow-up-7337 branch January 5, 2021 11:38
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.

3 participants