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

[MainUI] New sitemaps do not understand "visibility" & "valuecolor" #324

Closed
digitaldan opened this issue Sep 7, 2020 · 8 comments
Closed
Labels
main ui Main UI

Comments

@digitaldan
Copy link
Contributor

When trying to import my legacy sitemap using the New UI's sitemap editor, the lexer is choking on parsing the "valuecolor" attribute

Unexpected identifier token: "valuecolor". Instead, I was expecting to see one of the following:

An example sitemap entry using value color:

Text item=F2_Bedroom valuecolor=[==OFF="GRAY",!=OFF="green"]
Text label="Security [%s]" item=AlarmMode valuecolor=[==0="green", !=0="red"] icon="dd_security"
@ghys
Copy link
Member

ghys commented Sep 8, 2020

@ghys ghys added the main ui Main UI label Sep 24, 2020
@ghys ghys changed the title [MainUI] New sitemaps do not understand "valuecolor" [MainUI] New sitemaps do not understand "visibility" & "valuecolor" Nov 5, 2020
@stritti
Copy link

stritti commented Dec 23, 2020

I have the same issue transfere my sitemap from 2.5 to 3.0:
https://github.com/smart-swimmingpool/openhab-config/blob/master/sitemaps/pool.sitemap

@sirupflex
Copy link

The lack of support for visibility & valuecolor is quite painful and, in my opinion, contradicts a basic rule with OpenHAB. The basic rule is "either do everything in the UI or in text files", otherwise things get very messy and unpredictable with OpenHAB. With the lack of support for these important tags, you force people to maintain existing sitemaps in text files while pages are created in the UI, with no direct visibility of each other.

Please consider the implementation with high priority. In addition, I highly recommend the "visibility" feature to be implemented also for pages in general. If you want to have a user-friendly page that only displays relevant info based on a current status, you do need this feature. Today, you need to list a lot of items on a page but many of them might e.g. be displayed as "UNDEF" and therefore not adding any value to the page, confusing non-geek family members. Please think user-focused here, not technology-focused! OpenHAB 3 has fantastic potential, let's not forget the most important part, i.e. the interface to "general users", i.e. "the rest of the family", otherwise OpenHAB will remain an "IT geek tool". Thank you very much!

@ghys
Copy link
Member

ghys commented Apr 4, 2021

In addition, I highly recommend the "visibility" feature to be implemented also for pages in general.

https://www.openhab.org/docs/ui/building-pages.html#widgets-definition-usage

(also: https://twitter.com/youyuxi/status/1378409318102470663)

@sirupflex
Copy link

sirupflex commented Apr 4, 2021

Thanks for the hint regarding the widgets! Was not aware yet, i.e. I didn't understand that section when I looked at it the first time. It's a bit a painful process to edit the YAML directly but it works. I guess sooner or later this will come via UI as well.

Now, we "just" need this also for sitemaps and then we're fully aligned again. Then people can get rid of their text-based sitemaps and fully configure all the user UI part in the MainUI (which is a huge step forward compared to OH2.x!).

@airbone42
Copy link

+1 for this

@westward
Copy link

+1

1 similar comment
@MUDO86
Copy link

MUDO86 commented Aug 28, 2021

+1

@openhab openhab locked as spam and limited conversation to collaborators Aug 28, 2021
@ghys ghys closed this as completed in 72fe51e Oct 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
main ui Main UI
Projects
None yet
Development

No branches or pull requests

7 participants