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

[sitemap] Buttongrid with Button components available for main UI configuration #4377

Merged
merged 2 commits into from
Sep 28, 2024

Conversation

mherwege
Copy link
Contributor

@mherwege mherwege commented Sep 12, 2024

The syntax using nested Button components inside a Buttongrid component is currently not supported for configuration in the UI. This adds the necessary logic in core to make this possible.

@lolodomo

Related to #4223. This PR did not include the necessary part for UI sitemap configuration.

@mherwege mherwege requested a review from a team as a code owner September 12, 2024 20:50
Signed-off-by: Mark Herwege <[email protected]>
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @mherwege

Copy link
Member

@holgerfriedrich holgerfriedrich left a comment

Choose a reason for hiding this comment

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

LGTM

@holgerfriedrich holgerfriedrich merged commit 47284e5 into openhab:main Sep 28, 2024
5 checks passed
@holgerfriedrich holgerfriedrich added this to the 4.3 milestone Sep 28, 2024
@holgerfriedrich holgerfriedrich added the enhancement An enhancement or new feature of the Core label Sep 28, 2024
@mherwege mherwege deleted the buttongrid branch September 28, 2024 19:37
florian-h05 pushed a commit to openhab/openhab-webui that referenced this pull request Oct 4, 2024
Refs openhab/openhab-core#4377.

The most recent addition to the sitemap syntax allows defining buttons
nested as components inside the Buttongrid component.

---------

Signed-off-by: Mark Herwege <[email protected]>
stefan-hoehn pushed a commit to stefan-hoehn/openhab-webui that referenced this pull request Oct 5, 2024
Refs openhab/openhab-core#4377.

The most recent addition to the sitemap syntax allows defining buttons
nested as components inside the Buttongrid component.

---------

Signed-off-by: Mark Herwege <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature of the Core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants