Skip to content

refactor: solid settings (@miodec)#7816

Draft
Miodec wants to merge 64 commits intomasterfrom
solid-settings2
Draft

refactor: solid settings (@miodec)#7816
Miodec wants to merge 64 commits intomasterfrom
solid-settings2

Conversation

@Miodec
Copy link
Copy Markdown
Member

@Miodec Miodec commented Apr 12, 2026

Just don't look at the commit messages

@monkeytypegeorge monkeytypegeorge added frontend User interface or web stuff packages Changes in local packages labels Apr 12, 2026
modalClass?: string;
dialogClass?: string;
wrapperClass?: string;
//todo check if wrapper and dialog can be merged into one
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A new Todo was discovered. If it is not a priority right now,consider marking it for later attention.
todo check if wrapper and dialog can be merged into one

icon: "fa-pen",
}}
onClick={() => {
// todo: implement
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A new Todo was discovered. If it is not a priority right now,consider marking it for later attention.
todo: implement

icon: "fa-trash",
}}
onClick={() => {
// todo: implement
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A new Todo was discovered. If it is not a priority right now,consider marking it for later attention.
todo: implement

<Button
text="add preset"
onClick={() => {
// todo: implement
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A new Todo was discovered. If it is not a priority right now,consider marking it for later attention.
todo: implement

showSuccessNotification("Custom theme colors saved");
}}
/>
{/* //todo: save as new when logged in */}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A new Todo was discovered. If it is not a priority right now,consider marking it for later attention.
todo: save as new when logged in */}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff packages Changes in local packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants