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

Page tree broken after search is cleared #1296

Closed
timobrembeck opened this issue Mar 15, 2022 · 0 comments · Fixed by #1304
Closed

Page tree broken after search is cleared #1296

timobrembeck opened this issue Mar 15, 2022 · 0 comments · Fixed by #1304
Assignees
Labels
‼️ prio: high Needs to be resolved ASAP. 🐛 bug Something isn't working
Milestone

Comments

@timobrembeck
Copy link
Member

Describe the Bug

When the search form in the page tree is cleared, all pages are expanded by default, but the collapsing/expanding does not work as expected.

Maybe the source of the bug is the same as #1179 and it's not a browser caching problem after all? If so, maybe fixing this also fixes #1245 when we enable the browser cache again after the back-button.

Steps to Reproduce

  1. Go to the page tree
  2. Enter something into the seach bar and submit
  3. Clear the search via the "x"
  4. See broken page tree

Expected Behavior

The page tree should be fully collapsed and expanding should work normal

Actual Behavior

Everything is expanded and the tree becomes weird on toggling subpages

Additional Information

E.g. the following subtree:

Screenshot 2022-03-15 at 16-46-48 Integreat Redaktionssystem

Toggling the subpages only hides the middle level and the leaf pages are still visible

Screenshot 2022-03-15 at 16-46-58 Integreat Redaktionssystem

@timobrembeck timobrembeck added 🐛 bug Something isn't working ‼️ prio: high Needs to be resolved ASAP. labels Mar 15, 2022
@timobrembeck timobrembeck added this to the Version 1.1 milestone Mar 15, 2022
@timobrembeck timobrembeck self-assigned this Mar 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
‼️ prio: high Needs to be resolved ASAP. 🐛 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant