Skip to content

Conversation

@XDcsy
Copy link

@XDcsy XDcsy commented Dec 25, 2025

Implements #700, following the existing logic and UI patterns of the model star feature to ensure consistency.

BTW, I noticed that the current model star feature does not preserve the starred status after a model is renamed. In this PR, the starred status is preserved when a preset is renamed.

I did not modify the model-related behavior to keep this PR focused, but I would be happy to submit another one if needed.

Has been manually tested on Linux.

@mcmonkey4eva
Copy link
Member

This is the wrong approach -- Presets are already user-stored data, so they don't need a separate storage the way models do. They just need a boolean on the preset data

@hansneddyanto
Copy link

Hi @mcmonkey4eva Just wanted to clarify, the approach should be having an additional boolean column in t2i_presets (something like "IsStarred") to indicate whether a preset is starred or not ya instead of storing it in GenericData?

@mcmonkey4eva
Copy link
Member

not a "column", a "field", not in "t2i_presets" in T2IPreset, but yes.

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.

3 participants