Skip to content

Implement adding/removing timeouts #477

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 15 commits into
base: main
Choose a base branch
from
Draft

Implement adding/removing timeouts #477

wants to merge 15 commits into from

Conversation

@amycatgirl
Copy link
Contributor

@amycatgirl amycatgirl commented Oct 1, 2025 •
edited
Loading

Please make sure to check the following tasks before opening and submitting a PR

  • I understand and have followed the contribution guide
  • I have tested my changes locally and they are working as intended

Closes #287

Brief overview of the changes

  • Adds two new design components: Chip (MDUI) and InputTimePicker (Custom, needs more testing)
  • Adds TimeoutMemberModal
  • Adds two user context menu entries, "Timeout Member" and "Remove timeout"
  • Adds a timeout prop to MessageBox and a member prop to the message composer
  • Information is displayed to the timed out user as a notice in the MessageBox and is displayed to other users in the form of a badge

Still working out some kinks, but the modals/context menu entries work as expected.

@insertish insertish moved this to 🆕 Untriaged in Pull Request Overview Oct 1, 2025
@amycatgirl
Copy link
Contributor Author

amycatgirl commented Oct 1, 2025

fuck this was meant to be a draft, hold on

@amycatgirl amycatgirl marked this pull request as draft October 1, 2025 15:03
@amycatgirl
chore: add context menu entry for timeouts
33055b5 
Signed-off-by: Amy <[email protected]>
@amycatgirl
feat: chip component
5c8eb60 
Signed-off-by: Amy <[email protected]>
@amycatgirl
wip: start working on picker presets
f9f8385 
not sure how to signal the change to the input since the underlying
html inputs are handled separately, i should probably make it a option
within the component.

Signed-off-by: Amy <[email protected]>
@amycatgirl
chore: presets on time picker
ed8b433 
Signed-off-by: Amy <[email protected]>
@amycatgirl
chore: add remove timeout button to UserContextMenu
9b0bc3f 
Signed-off-by: Amy <[email protected]>
@amycatgirl
chore: make currently selected preset more visible
aafac07 
Signed-off-by: Amy <[email protected]>
@amycatgirl
chore: allow supporting text to be translated
45b1116 
Signed-off-by: Amy <[email protected]>
@amycatgirl amycatgirl force-pushed the feat/timeouts-again branch from 1b2edbc to 45b1116 Compare October 1, 2025 15:25
@amycatgirl
chore: clarify why translations are there
0181f0e 
whyyyyyyyy

Signed-off-by: Amy <[email protected]>
@amycatgirl
feat: show timeout information on MessageBox
0098a23 
Signed-off-by: Amy <[email protected]>
@amycatgirl
feat: show timeout information in message as badge
d59c71b 
i swear this was implemented before, oh well. it's implemented now 👍

Signed-off-by: Amy <[email protected]>
@amycatgirl amycatgirl marked this pull request as ready for review October 1, 2025 16:43
@insertish insertish marked this pull request as draft October 10, 2025 14:20
@insertish
Copy link
Contributor

insertish commented Oct 10, 2025

Marked as draft

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: timeout members

2 participants

@amycatgirl @insertish