Skip to content

Add checked item section to shopping-list-card#2005

Merged
balloob merged 8 commits intohome-assistant:devfrom
iantrich:shopping-checked
Nov 9, 2018
Merged

Add checked item section to shopping-list-card#2005
balloob merged 8 commits intohome-assistant:devfrom
iantrich:shopping-checked

Conversation

@iantrich
Copy link
Copy Markdown
Member

@iantrich iantrich commented Nov 7, 2018

lovelace_shopping_list_card

@ghost ghost assigned iantrich Nov 7, 2018
@ghost ghost added the in progress label Nov 7, 2018
@balloob
Copy link
Copy Markdown
Member

balloob commented Nov 7, 2018

The reason localize doesn't work is because you override the hass setter, meaning that Lit doesn't know it has changed. You should just remove the setter for _hass and render based on this.hass

@fabaff
Copy link
Copy Markdown
Member

fabaff commented Nov 7, 2018

Documentation: home-assistant/home-assistant.io#7346

}
`
)
: html``
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

""

}
}

private _completeItem(ev): void {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Param Type

);
}

private _saveEdit(ev): void {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Param Type

Copy link
Copy Markdown
Member Author

@iantrich iantrich Nov 9, 2018

Choose a reason for hiding this comment

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

ech. I don't know how to handle this. It's a MouseEvent...but the target is then a paper-checkbox type with an additional field itemId that I added...

Copy link
Copy Markdown
Member Author

@iantrich iantrich Nov 9, 2018

Choose a reason for hiding this comment

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

actually _saveEdit would be a KeyboardEvent or a MouseEvent...but only applies when you either click out or hit enter...I don't know what to do...

@zsarnett
Copy link
Copy Markdown
Contributor

zsarnett commented Nov 8, 2018

Also need to rebase. Or resolve Merge Conflicts

@balloob balloob merged commit 9c2b85d into home-assistant:dev Nov 9, 2018
@ghost ghost removed the in progress label Nov 9, 2018
@iantrich iantrich deleted the shopping-checked branch November 16, 2018 19:17
@github-actions github-actions bot locked and limited conversation to collaborators Jul 7, 2022
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.

5 participants