Skip to content

Conversation

@hehoon
Copy link
Collaborator

@hehoon hehoon commented Dec 13, 2025

I have JIRA issue created

  • branch and/or PR name(s) includes JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected
  • FLP integration tests were ran successful

Changes:

  • The tree on the object tree page now persists its expanded node state in local storage.

Note:

  • The estimated storage size (in bytes) is approximately: A * (N + 7) * 2
    • A is the number of expanded (opened) nodes
    • N is the average character length of expanded (opened) node names
  • The storing of tree node states will break if two identical names are stored directly under the same parent. This limitation already existed in the previous implementation and is unchanged by the current persistence mechanism.
  • A temporary workaround was added to handle local storage resets in tests, since some tests are not fully isolated and depend on shared state. Ideally, all tests should be isolated, with local storage and page state fully reset between each test.

@hehoon hehoon marked this pull request as ready for review December 15, 2025 13:48
@hehoon hehoon requested a review from graduta as a code owner December 15, 2025 13:48
Copy link
Member

@graduta graduta left a comment

Choose a reason for hiding this comment

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

The features works very well, good job!

I believe there is one case that you missed though. If the user clicks on the Collapse all button from the header of the page, the state of the objects is not saved, so a refresh of the page will reopen the previous state

@hehoon
Copy link
Collaborator Author

hehoon commented Jan 5, 2026

The features works very well, good job!

I believe there is one case that you missed though. If the user clicks on the Collapse all button from the header of the page, the state of the objects is not saved, so a refresh of the page will reopen the previous state

This should be solved as of 5b80e3c

EDIT: Also see: 221980e

@hehoon hehoon marked this pull request as draft January 5, 2026 13:58
@graduta graduta closed this Jan 5, 2026
@graduta graduta reopened this Jan 5, 2026
@hehoon hehoon requested a review from graduta January 5, 2026 14:33
@hehoon hehoon marked this pull request as ready for review January 5, 2026 14:33
graduta
graduta previously approved these changes Jan 6, 2026
@hehoon hehoon requested a review from graduta January 6, 2026 15:50
@graduta graduta merged commit 0a6b618 into dev Jan 6, 2026
10 checks passed
@graduta graduta deleted the feature/QCG/OGUI-689/tree-stores-state-of-nodes-in-storage branch January 6, 2026 16:00
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.

3 participants