Skip to content

Fix apply/save issue#91

Open
Osayi-ANL wants to merge 2 commits into
mainfrom
dev
Open

Fix apply/save issue#91
Osayi-ANL wants to merge 2 commits into
mainfrom
dev

Conversation

@Osayi-ANL

@Osayi-ANL Osayi-ANL commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator
  • feat: apply/save change tracking, copy/paste, and close-event prompt
  • fix: initialize _structural_changed in init and reset on save/clear to prevent ERROR for not initialized

Test Plan:

  • Apply/Save and Clear disabled, before any entry
  • Open the config change a pv -- there is an indicator of a change and apply and clear is enabled and a prompt
  • Right click, Copy a key value and paste in a section
  • Make an edit and then try to close before saving there is a prompt to save or discard edits

Osayi-ANL added 2 commits June 25, 2026 11:26
@Osayi-ANL
feat: apply/save change tracking, copy/paste, and close-event prompt
4d800f8 
fix: remove premature _save_tree_to_active_profile calls on structural edits

style: buttonApplySave styled via theme_colors tokens

resolves #83
@Osayi-ANL
fix: initialize _structural_changed in __init__ and reset on save/cle…
7d0c364 
…ar to prevent ERROR for not initialized

fix: rewired the clear button to the refresh so they operate the same
@Osayi-ANL Osayi-ANL requested a review from pecomyint June 25, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pecomyint pecomyint pecomyint approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Osayi-ANL @pecomyint