Skip to content

feat: Implement an UI for commands#129

Open
Rosy-iso wants to merge 31 commits intoSableClient:devfrom
Rosy-iso:feat/ui-for-commands
Open

feat: Implement an UI for commands#129
Rosy-iso wants to merge 31 commits intoSableClient:devfrom
Rosy-iso:feat/ui-for-commands

Conversation

@Rosy-iso
Copy link
Contributor

@Rosy-iso Rosy-iso commented Mar 10, 2026

Description

Adds an interface that replicates the behavior of the following commands:

  • /myroomavatar
  • /myroomnick
  • /color
  • /scolor
  • /pronoun
  • /spronoun
  • /font
  • /sfont

The original commands will still remain and be usable however.
(In fact the interface calls the commands in the background, so the commands should not be changed.)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@Rosy-iso Rosy-iso changed the title Feat/Implement an UI for commands feat: Implement an UI for commands Mar 10, 2026
@Rosy-iso Rosy-iso marked this pull request as ready for review March 10, 2026 17:31
@Rosy-iso Rosy-iso requested a review from a team March 10, 2026 17:31
@7w1
Copy link
Member

7w1 commented Mar 10, 2026

Only issue I see is that there's no feedback for if the user has permission to change their pronouns/color/font in a space. Could disable those fields similar based on whether or not the user has permission (look at the code for the toggles and disable if the toggles are disabled, or similar).

@Rosy-iso
Copy link
Contributor Author

oh right, i forgot those could be disabled
I'll make them disable when the user doesn't have permission

@Rosy-iso
Copy link
Contributor Author

should be all good now

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.

2 participants