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

UI - sidebar link, wrong highlighting #1966

Closed
ifs-net opened this issue Jul 7, 2023 · 1 comment · Fixed by #1967
Closed

UI - sidebar link, wrong highlighting #1966

ifs-net opened this issue Jul 7, 2023 · 1 comment · Fixed by #1967
Labels
bug Something isn't working main ui Main UI

Comments

@ifs-net
Copy link

ifs-net commented Jul 7, 2023

I have different pages (charts) that can be accessed in the UI.

e.g. "pv_wechselricher" and "pv_wechselricher_temperatur"

Current Behavior

when I select the first one, the correct link is highlighted. selecting the second page, the first is also highlighted.

Expected Behavior

only the selected page is highlighted

Steps to Reproduce (for Bugs)

see screenshots for the data of the different pages

Your Environment

openhab 3.4.3 (docker-container synology)

image

image

image
You can see that the first page is also highlighted

image

@ifs-net ifs-net added the bug Something isn't working label Jul 7, 2023
@J-N-K
Copy link
Member

J-N-K commented Jul 16, 2023

@openhab/webui-maintainers This is an UI issue.

@J-N-K J-N-K added the UI label Jul 16, 2023
@kaikreuzer kaikreuzer transferred this issue from openhab/openhab-core Jul 16, 2023
@lolodomo lolodomo added main ui Main UI and removed UI labels Jul 17, 2023
florian-h05 pushed a commit that referenced this issue Jul 18, 2023
Fixes #1966.

Happens when you have pages prefixed with the name of another page.

Signed-off-by: Yannick Schaus <github@schaus.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working main ui Main UI
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants