[#68031] Fix toggleable fieldsets

[myabc]

Ticket

https://community.openproject.org/wp/68031

What are you trying to accomplish?

Ensure event listeners are only attached once - on turbo:load.

Screenshots

What approach did you choose and why?

This partially reverts changes made in #19495

Merge checklist

• Added/updated tests
• Added/updated documentation in Lookbook (patterns, previews, etc)
• Tested major browsers (Chrome, Firefox, Edge, ...)

[github-actions]

Caution

The provided work package version does not match the core version

Details:

• Work package URL: https://community.openproject.org/wp/68031
• Work package version: not set
• Core version: 16.6.0

Please make sure that:

• The work package version OR your pull request target branch is correct

[HDinger]

I am not too sure about this change.. Yes, it fixes the issue but looking at the commit message of the deleted line, it was written:

[#65298] Fix broken top bar menus on form errors

Menus in the top bar were not properly initialised after unsuccessful
form submissions that result in an 422 Unprocessable Entity response
via Turbo Drive.

This commit updates Angular-Turbo Drive integration code to listen to
turbo:render in addition to turbo:load. turbo:load is only fired
on Turbo Drive visits. However, a form submission response that is not
a 303 Redirect is not considered a visit by Turbo Drive.

See: hotwired/turbo#85
See also: https://turbo.hotwired.dev/handbook/drive#redirecting-after-a-form-submission

So I guess, this might introduce a regression?

Further, intependent of that: How does tubro interfere with the toggling mechanism at all? And did you try to replace that with the CollapsibleSection component we have in Primer?