Skip to content

feat: Voice Call Button in header#1025

Open
Pecacheu wants to merge 4 commits intostoatchat:mainfrom
Pecacheu:voicebtn
Open

feat: Voice Call Button in header#1025
Pecacheu wants to merge 4 commits intostoatchat:mainfrom
Pecacheu:voicebtn

Conversation

@Pecacheu
Copy link
Copy Markdown
Contributor

@Pecacheu Pecacheu commented Mar 13, 2026

Split off from #835

Replaces the obnoxious Voice Call preview card with a button in the header for DMs

But retains it in server voice channels (as requested by Insert)

Fixes #541

@Pecacheu Pecacheu changed the title Voice Call Button in header feat: Voice Call Button in header Mar 13, 2026
@Pecacheu
Copy link
Copy Markdown
Contributor Author

Annnnd of course there's merge conflicts in the strings even tho I didn't add any new ones. Why not?

@Dissssy
Copy link
Copy Markdown

Dissssy commented Mar 14, 2026

I love the stoat devs but i'm pretty sure this is like the third open PR for this specific ui feature in the last month lol

@Pecacheu
Copy link
Copy Markdown
Contributor Author

Pecacheu commented Mar 14, 2026

@Dissssy Technically it was the first, but they told me to split off unrelated things from my mobile UI mega-PR into their own PRs. I ended up fixing this early on as it was way more annoying on a small phone screen.

@Pecacheu Pecacheu added the ux review This needs UX review label Mar 14, 2026
@ThomasJRyan ThomasJRyan requested a review from infi March 15, 2026 05:03
@Pecacheu Pecacheu requested review from a team as code owners March 21, 2026 15:11
@Pecacheu Pecacheu force-pushed the voicebtn branch 2 times, most recently from ea1c3a0 to a94f16f Compare March 22, 2026 01:42
@Pecacheu Pecacheu force-pushed the voicebtn branch 2 times, most recently from 73c2933 to efbfcab Compare March 31, 2026 18:14
@gabrielfordevelopment gabrielfordevelopment added the fluff Minor improvement / quality of life change label Apr 22, 2026
@gabrielfordevelopment
Copy link
Copy Markdown
Contributor

Can you provide a full screenshot?

@Pecacheu
Copy link
Copy Markdown
Contributor Author

@gabrielfordevelopment Sure, here you go

Out of call on server (unchanged)

Out of call on DM:

In call on DM:

@gabrielfordevelopment
Copy link
Copy Markdown
Contributor

Thank you so much for the provided media files.
It looks a bit weird in voice channels that there is a duplicated button that does the same thing.
image

@Pecacheu
Copy link
Copy Markdown
Contributor Author

Thank you so much for the provided media files. It looks a bit weird in voice channels that there is a duplicated button that does the same thing.

I think it looks fine that way, but we could make it hide when in a call. Question for UI/UX

AvaLilac pushed a commit to AvaLilac/Ava-s-AviaClient-Plugins that referenced this pull request Apr 24, 2026
Comment thread packages/client/src/interface/channels/ChannelHeader.tsx Outdated
@infi
Copy link
Copy Markdown
Contributor

infi commented May 1, 2026

Resolutions on disagreements above:

  • The call card should always be visible whenever there is an ongoing call in a channel
  • There must not be a second "end call" button on the screen, that feature is part of the extended call card

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

Labels

fluff Minor improvement / quality of life change ux review This needs UX review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: hide/toggle "Join Call" card

5 participants