fix: update config panel when deleting a connection

[Jondyr]

Description

This PR changes the logic for tracking which element is selected in process editor. Using the event from diagramjs selection.changed instead of trying to sync our selection through shape.add, shape.remove, connection.remove and element.click.
This solves some other issues like for example: the Ctrl-F finder not updating selected element in the config panel.

Related Issue(s)

• Config panel not updated after deleting an association #13135

Verification

• Your code builds clean without any errors or warnings
• Manual testing done (required)
• Relevant automated test added (if you find this hard, leave it and we'll help out)

Documentation

• User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.99%. Comparing base (61e0ab2) to head (1787130).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #13437   +/-   ##
=======================================
  Coverage   93.99%   93.99%           
=======================================
  Files        1456     1456           
  Lines       20106    20108    +2     
  Branches     2438     2438           
=======================================
+ Hits        18899    18901    +2     
  Misses        954      954           
  Partials      253      253           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[standeren]

Nice solution! 🤩
I only miss a more descriptive test-case for the triggering of selection.changed event.
And I have a suggestion removing the oldSelection variable since it does not look like we need it (yet), but I also see the pros of keeping it, so that one is up to you 😬

[standeren]

Nice changes!
But can we avoid using any for the currentEventType?

[Jondyr]

Nice changes! But can we avoid using any for the currentEventType?

I'm not sure what would make sense to type it as, as I'm unsure if bpmn-js or diagram-js exports typedefiniton for their events, i find the documentation very hard to navigate/search so it might be there somewhere and i've just not found it yet.

[TomasEng]

Testet OK.