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

Feat/baylabels #242

Merged
merged 33 commits into from
Mar 12, 2025
Merged

Feat/baylabels #242

merged 33 commits into from
Mar 12, 2025

Conversation

pwinkelholzS1
Copy link
Collaborator

@pwinkelholzS1 pwinkelholzS1 commented Jan 9, 2025

🗒 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.

  • Ticket-ACs are checked and met
  • GitHub labels are assigned
  • Git Ticket / issue is linked with PR
  • Designated PR Reviewers are set
  • Tested by another developer
  • Changelog updated
  • Documentation updated (check Documentation guidelines for further reading )
  • All comments in the PR are resolved / answered

🔦 Useful commits

You can add specific commits which are worth taking a look into

# 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
@pwinkelholzS1 pwinkelholzS1 linked an issue Feb 12, 2025 that may be closed by this pull request
4 tasks
# Conflicts:
#	packages/uilib/src/lib/components/diagram/nodes.ts
#	packages/uilib/src/lib/plugins/communication-explorer/_func-layout-calculation/node-layout-ieds.ts
(cherry picked from commit ce31446)
(cherry picked from commit 2f5d824)
(cherry picked from commit 8347ad2)
(cherry picked from commit 1f7e648)
@pwinkelholzS1 pwinkelholzS1 marked this pull request as ready for review February 19, 2025 11:29
Copy link
Collaborator

@michelguerin michelguerin left a 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

@pwinkelholzS1
Copy link
Collaborator Author

i found and fixed 2 bugs that caused strange behavior, it should be working as intended now

Copy link
Collaborator

@trusz trusz left a 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

@pwinkelholzS1 pwinkelholzS1 merged commit 4c94ec5 into main Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show Bays in diagram overview (Baylabels)
3 participants