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

Configuration of context labels in Equipment and Properties card of homepage tabs. #1395

Merged
merged 1 commit into from
May 29, 2022

Conversation

tarag
Copy link
Contributor

@tarag tarag commented May 20, 2022

Addition of configuration options available are the path in model (joining parents label), the parent label, the item name or nothing. The possibility to trim beginning or end of path is also configurable.
If nothing is selected, will display the footer defined in metadata (if any).

This should hopefully satisfy users who complained about the lack of configurability when the improvements of PR #1264 were introduced.

…operties card of homepage tabs.

Options available are the path in model (joining parents label), the parent label, the item name or nothing. The possibility to trim beginning or end of path is also configurable.
If nothing is selected, will display the footer defined in metadata (if any).

Signed-off-by: Gautier Taravella <tarag@mailbox.org>
@tarag tarag requested a review from a team as a code owner May 20, 2022 15:59
@relativeci
Copy link

relativeci bot commented May 20, 2022

Job #402: Bundle Size — 10.71MB (+0.04%).

1fd5582 vs bb5e51c

Changed metrics (3/10)
Metric Current Baseline
Initial JS 1.67MB(+0.1%) 1.67MB
Cache Invalidation 23.64% 21.32%
Duplicate Code 1.99%(+0.51%) 1.98%
Changed assets by type (1/7)
            Current     Baseline
JS 8.65MB (+0.05%) 8.65MB

View Job #402 report on app.relative-ci.com

Copy link
Member

@ghys ghys left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM.
I don't remember if I already told you this, but I think it could be great to have these settings on a per-card basis too, because deviating from the tab's default might make sense in some cases (like no context in some cards if the label is enough).

@ghys ghys merged commit c4493ee into openhab:main May 29, 2022
@ghys ghys added enhancement New feature or request main ui Main UI labels May 29, 2022
@ghys ghys added this to the 3.3 milestone May 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request main ui Main UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants