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

feat: allow using live directive in LitRenderer #6068

Merged
merged 4 commits into from
Feb 20, 2024

Conversation

DiegoCardoso
Copy link
Contributor

Description

Inject the live directive into the closure where the render is executed.

It's beneficial for cases where components can have their state changed by users to reset their value when the render is called.

Inject the `live` directive into the closure where the render is executed.

It's particular useful for cases where components that can have their state changed by
users to reset their value when the render is called.
@DiegoCardoso DiegoCardoso force-pushed the feat/lit-renderer/add-live-directive branch from 4c6868a to 62654a8 Compare February 20, 2024 12:31
@DiegoCardoso DiegoCardoso enabled auto-merge (squash) February 20, 2024 14:15
Copy link

sonarcloud bot commented Feb 20, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

idea Catch issues before they fail your Quality Gate with our IDE extension SonarLint SonarLint

@DiegoCardoso DiegoCardoso merged commit deb0639 into main Feb 20, 2024
4 of 5 checks passed
@DiegoCardoso DiegoCardoso deleted the feat/lit-renderer/add-live-directive branch February 20, 2024 15:05
vaadin-bot pushed a commit that referenced this pull request Feb 20, 2024
* feat: allow using `live` directive in LitRenderer

Inject the `live` directive into the closure where the render is executed.
It's beneficial for cases where components can have their state changed by users to reset their value when the render is called.
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 24.4.0.alpha5 and is also targeting the upcoming stable 24.4.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants