-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/baylabels #242
Feat/baylabels #242
Conversation
# Conflicts: # packages/core/src/use-cases/uc-communication-information.ts # packages/uilib/src/lib/components/diagram/nodes.ts # packages/uilib/src/lib/plugins/communication-explorer/_func-layout-calculation/node-layout-connections.ts # packages/uilib/src/lib/plugins/communication-explorer/_func-layout-calculation/node-layout-ieds.ts # packages/uilib/src/lib/plugins/communication-explorer/_store-view-filter/selected-filter-store-functions.ts # packages/uilib/src/lib/plugins/communication-explorer/_store-view-filter/selected-filter-store.ts # packages/uilib/src/lib/plugins/communication-explorer/sidebar/ied-accordion/ied-accordion.svelte # packages/uilib/src/lib/plugins/communication-explorer/telemetry-view/telemetry-view.svelte
# Conflicts: # packages/uilib/src/lib/components/diagram/nodes.ts # packages/uilib/src/lib/plugins/communication-explorer/_func-layout-calculation/node-layout-ieds.ts # packages/uilib/src/lib/plugins/communication-explorer/sidebar/ied-accordion/ied-accordion.svelte
# Conflicts: # packages/uilib/src/lib/components/diagram/nodes.ts # packages/uilib/src/lib/plugins/communication-explorer/_func-layout-calculation/node-layout-ieds.ts
# Conflicts: # packages/uilib/src/lib/components/diagram/ied-element/ied-element.svelte
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The overall flow seems inconsistent : selecting the label in different places doesn't produce the same effect. Can you check if this is the expected behavior ?
Screen.Recording.2025-02-21.at.08.12.51.mov
packages/uilib/src/lib/plugins/communication-explorer/telemetry-view/telemetry-view.svelte
Show resolved
Hide resolved
i found and fixed 2 bugs that caused strange behavior, it should be working as intended now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please check if it does not affect the documentation tool before merging
🗒 Description
adding baylabels + ability to toggle between baycontainers & baylabels
clicking on a baylable selects all corresponding IEDs, similar to sidebar bay selection
📷 Demo screen recording or Screenshots
OpenSCD.Mozilla.Firefox.2025-02-20.16-47-17.mp4
📋 Checklist
You may remove tasks that do not make sense in this PR.
🔦 Useful commits
You can add specific commits which are worth taking a look into