Skip to content

Update Edit Footer for Cards#4752

Merged
balloob merged 3 commits into
home-assistant:devfrom
zsarnett:edit-options
Feb 6, 2020
Merged

Update Edit Footer for Cards#4752
balloob merged 3 commits into
home-assistant:devfrom
zsarnett:edit-options

Conversation

@zsarnett
Copy link
Copy Markdown
Contributor

@zsarnett zsarnett commented Feb 4, 2020

Proposed change

Updates the Style of the Edit Options to better flow with the current Card. This fixes the Material Design issues as there is not Shadow footer

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

image

⬇️

(old image with out HA-CARD) ❗️
image

(New Image with HA-CARD) ❗️
image

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.

Copy link
Copy Markdown
Member

@balloob balloob left a comment

Choose a reason for hiding this comment

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

So good!

Comment thread src/panels/lovelace/components/hui-card-options.ts Outdated
@zsarnett zsarnett requested a review from balloob February 5, 2020 19:38
ha-card {
border-top-right-radius: 0;
border-top-left-radius: 0;
box-shadow: rgba(0, 0, 0, 0.14) 0px 2px 2px 0px,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is this still needed ? Isn't this built into the card ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is a different shadow from the Ha-card. It shifts the shadow down so there is no shadow on the overlapping section

@balloob
Copy link
Copy Markdown
Member

balloob commented Feb 6, 2020

ok to merge when last comment answered

@balloob balloob merged commit 2e47aa1 into home-assistant:dev Feb 6, 2020
@lock lock Bot locked and limited conversation to collaborators Feb 8, 2020
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