Overflow fix

[yosilevy]

Breaking change

Proposed change

Fixes overflow problems in RTL

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:

• Documentation added/updated for www.home-assistant.io

[wendevlin]

Hey @yosilevy,

when there is no graph you hide the date selection:

[home-assistant]

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

[yosilevy]

@wendevlin Thanks for finding this. Fixed.

[wendevlin]

@yosilevy thank you,

But there is still an issue on mobile, when calendar controls are higher then the screen.

Can you maybe specify the bug you try to solve in more details? With a video or screenshot where the issue is.

[bramkragten]

Set the overflow on a higher element, like in ha-top-app-bar-fixed

[yosilevy]

@bramkragten thanks. Done with some modifications which make it work.
@wendevlin thanks for the catch. I checked this one inside web debugger for mobile and web and it seems OK. I hope you'll see same. Thanks.

[wendevlin]

👍 Thank you!