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

Extend "iconcolor" attribute to all sitemap elements #3403

Merged
merged 2 commits into from
Feb 26, 2023

Conversation

lolodomo
Copy link
Contributor

This attribute was already available in syntax for the Image element but was probably not used by any UI.
It is now available for any sitemap element and allows defining a color for the widget icon based on conditional statements on the item value.

Signed-off-by: Laurent Garnier lg.hc@free.fr

This attribute was already available in syntax for the Image element but
was probably not used by any UI.
It is now available for any sitemap element and allows defining a color
for the widget icon based on conditional statements on the item value.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
@lolodomo lolodomo requested a review from a team as a code owner February 25, 2023 17:33
Copy link
Member

@J-N-K J-N-K left a comment

Choose a reason for hiding this comment

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

Thanks, I have left one comment. Otherwise LGTM.

@J-N-K J-N-K added the enhancement An enhancement or new feature of the Core label Feb 25, 2023
@J-N-K J-N-K changed the title iconcolor attribute added to all sitemap elements Extend "iconcolor" attribute to all sitemap elements Feb 25, 2023
@J-N-K J-N-K added the sitemap label Feb 25, 2023
Signed-off-by: Laurent Garnier <lg.hc@free.fr>
Copy link
Member

@J-N-K J-N-K left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@J-N-K J-N-K merged commit f2b8fa7 into openhab:main Feb 26, 2023
@lolodomo lolodomo deleted the sitemap_model_iconcolor branch February 26, 2023 18:52
@lolodomo
Copy link
Contributor Author

lolodomo commented Mar 5, 2023

For information
@openhab/webui-maintainers
@openhab/android-maintainers
@openhab/ios-maintainers

florian-h05 pushed a commit to openhab/openhab-webui that referenced this pull request Mar 6, 2023
splatch pushed a commit to ConnectorIO/copybara-hab-core that referenced this pull request Jul 12, 2023
* iconcolor attribute added to all sitemap elements

This attribute was already available in syntax for the Image element but
was probably not used by any UI.
It is now available for any sitemap element and allows defining a color
for the widget icon based on conditional statements on the item value.

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
GitOrigin-RevId: f2b8fa7
@wborn wborn added this to the 4.0 milestone Jul 15, 2023
@openhab-bot
Copy link
Collaborator

This pull request has been mentioned on openHAB Community. There might be relevant details there:

https://community.openhab.org/t/android-app-iconcolor-on-sitemap-is-buggy/149012/15

maniac103 added a commit to maniac103/openhab.android that referenced this pull request Sep 5, 2023
Support for that field was added in openhab/openhab-core#3403

Fixes openhab#3466

Signed-off-by: Danny Baumann <dannybaumann@web.de>
mueller-ma pushed a commit to openhab/openhab-android that referenced this pull request Sep 5, 2023
Support for that field was added in openhab/openhab-core#3403

Fixes #3466

Signed-off-by: Danny Baumann <dannybaumann@web.de>
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 sitemap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants