Skip to content

518 allow to persist cell select mode per file#534

Merged
Hirogen merged 3 commits intoDevelopmentfrom
518-allow-to-persist-cell-select-mode-per-file
Mar 3, 2026
Merged

518 allow to persist cell select mode per file#534
Hirogen merged 3 commits intoDevelopmentfrom
518-allow-to-persist-cell-select-mode-per-file

Conversation

@Hirogen
Copy link
Copy Markdown
Collaborator

@Hirogen Hirogen commented Mar 3, 2026

This pull request introduces support for persisting and restoring the cell selection mode for log windows, ensuring that user preferences for selection (cell vs. row) are saved and restored across sessions. The main changes involve updating the persistence logic, the UI state management, and related tests.

Persistence and restoration of cell selection mode:

  • Added a new property CellSelectMode to the PersistenceData class to store the user's cell selection mode preference.
  • Updated the GetPersistenceData method in LogWindow.cs to save the current cell selection mode to persistence data.
  • Modified both LoadPersistenceOptions and LoadPersistenceData in LogWindow.cs to restore the cell selection mode from the persisted data when loading a session. [1] [2]

UI and state management improvements:

  • Enhanced the SetCellSelectionMode method to accept an updateGUI parameter, allowing the GUI to be updated when the selection mode changes, and triggering a GUI state update as needed. [1] [2]

Test coverage:

  • Extended the RoundTrip_WithApplicationStartupDir_PreservesAllData test to include assertions for the new CellSelectMode property, ensuring persistence logic works as intended. [1] [2]

@Hirogen Hirogen linked an issue Mar 3, 2026 that may be closed by this pull request
@Hirogen Hirogen merged commit 1387142 into Development Mar 3, 2026
1 check passed
@Hirogen Hirogen deleted the 518-allow-to-persist-cell-select-mode-per-file branch March 3, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow to persist Cell select mode per file

1 participant