Skip to content

[OP-19878] Always display tree and check all button - #24757

Open
klaustopher wants to merge 1 commit into
release/17.7from
bugfix-OP-19878
Open

[OP-19878] Always display tree and check all button#24757
klaustopher wants to merge 1 commit into
release/17.7from
bugfix-OP-19878

Conversation

@klaustopher

Copy link
Copy Markdown
Contributor

Ticket

https://community.openproject.org/projects/OP/work_packages/OP-19878

What are you trying to accomplish?

  • Fix an error where the enable for all projects action was trying to render a turbo_stream response which it could not find the template for
  • Fix an issue with the treeview, where the click was not properly propagating
  • Always display the tree view, even if the "enable for all projects" is on, so that when the type is used in any work package, we can still disable it on a single project that is not using it.

Screenshots

Bildschirmfoto 2026-08-14 um 14 59 47

What approach did you choose and why?

Merge checklist

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

@github-actions

Copy link
Copy Markdown

Caution

The provided work package version does not match the core version

Details:

Please make sure that:

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

@klaustopher klaustopher changed the title Always display tree and check all button [OP-19878] Always display tree and check all button Aug 14, 2026
Comment on lines +54 to +61
// Primer stops the propagation of clicks on sub tree nodes, so listening for clicks would
// miss every parent project.
this.observer = new MutationObserver(() => this.updateSelectedProjects());
this.observer.observe(this.treeViewTarget, {
childList: true,
subtree: true,
attributeFilter: ['aria-checked'],
});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this relate to opf/primer_view_components#515 ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants