Skip to content

Issue 59: Prioritize User Presets and Verify Save Behavior#63

Open
google-labs-jules[bot] wants to merge 6 commits intomainfrom
fix-issue-59-7379445118819867893
Open

Issue 59: Prioritize User Presets and Verify Save Behavior#63
google-labs-jules[bot] wants to merge 6 commits intomainfrom
fix-issue-59-7379445118819867893

Conversation

@google-labs-jules
Copy link
Contributor

Implemented Issue #59:

  1. User presets are now displayed at the top of the dropdown list, immediately after the "Default" built-in preset. This makes them much more accessible for users who frequently switch between their custom settings.
  2. Verified that the "Save Current Config" button in the GUI already handles updating the currently selected user preset without requiring a rename, addressing the second part of the issue.
  3. Updated the preset insertion logic (Add, Duplicate, Import) to ensure new presets are placed in the user section at the top rather than appended to the end.
  4. Incremented application version to v0.7.16 and documented changes in CHANGELOG_DEV.md.
  5. Added comprehensive unit tests in tests/test_issue_59.cpp and updated existing tests to ensure robustness against the new ordering logic.

PR created automatically by Jules for task 7379445118819867893 started by @coasting-nc

- Reordered Config::LoadPresets to load user presets immediately after 'Default'.
- Updated AddUserPreset, DuplicatePreset, and ImportPreset to insert at the user section.
- Verified that 'Save Current Config' correctly updates the selected user preset.
- Incremented version to 0.7.16 and updated changelog.
- Added TDD tests for preset ordering and insertion logic.
- Updated existing tests to handle new preset ordering.
@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

google-labs-jules bot and others added 5 commits February 6, 2026 09:29
- Implemented user preset reordering: user presets now appear at the top of the dropdown list, immediately after "Default".
- Refactored preset management into a dedicated PresetRegistry singleton class.
- Verified "Save Current Config" correctly updates existing user presets.
- Restored parameter clamping and migration logic for robustness.
- Updated version to 0.7.17 and updated CHANGELOG_DEV.md.
- Expanded test suite with 448 passing tests, covering the new architecture and reordering logic.
- Implemented user preset reordering in dropdown list (Issue #59).
- Refactored preset management into dedicated PresetRegistry class.
- Updated version to 0.7.19 to account for latest main branch merges.
- Documented changes in CHANGELOG_DEV.md as versions 0.7.18 and 0.7.19.
- Updated implementation plans to match the final versioning scheme.
- Verified all 448 tests pass.
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.

1 participant