Skip to content

fix: standup feedback round#5990

Merged
idoshamun merged 3 commits into
mainfrom
fix/standup-feedback
May 5, 2026
Merged

fix: standup feedback round#5990
idoshamun merged 3 commits into
mainfrom
fix/standup-feedback

Conversation

@idoshamun
Copy link
Copy Markdown
Member

@idoshamun idoshamun commented May 5, 2026 •
edited by github-actions Bot
Loading

Summary

  • Tighten chat message spacing on both desktop and mobile.
  • Float the reaction tray on hover so empty messages no longer reserve a row, and flip the placement for the topmost message so it isn't hidden behind the side-panel tabs.
  • Always show all quick reactions in the floating tray; reacting again with the same emoji now toggles it off.
  • Drop per-emoji tooltips and make the reactions toolbar opaque.
  • Click on a focused video tile now defocuses it.
  • Prefer the system default mic/camera (deviceId === 'default') instead of the first enumerated device.
  • Raise the default free-for-all speaker limit to 100.

Test plan

  • Chat: messages render tightly on desktop & mobile; hover panel floats on desktop, hides on mobile.
  • Hover panel: reach all buttons without losing hover; no overlap with prior message.
  • Topmost message hover panel renders below the message and is fully visible.
  • Re-clicking an already-reacted emoji in the floating tray removes that reaction.
  • Reactions toolbar over the controls is opaque, no per-emoji tooltips.
  • Focusing a tile and clicking it again defocuses.
  • Default mic/camera selected on join; switching still works.
  • Free-for-all standup default speaker limit shows 100.

Preview domain

https://fix-standup-feedback.preview.app.daily.dev

@idoshamun
fix: standup feedback round
99cd1cf 
- Tighten chat message spacing.
- Float reaction tray on hover so empty messages no longer reserve a row;
  flip placement for the topmost message to avoid the side-panel tabs.
- Always show all quick reactions in the floating tray and toggle when
  re-clicking an already-reacted emoji.
- Make reactions toolbar opaque and drop per-emoji tooltips.
- Click on a focused video tile now defocuses it.
- Prefer the system default mic/camera (deviceId === 'default') over the
  first enumerated device.
- Raise the default free-for-all speaker limit to 100.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 5, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
daily-webapp Ready Ready Preview May 5, 2026 2:41pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
storybook Ignored Ignored May 5, 2026 2:41pm

Request Review

@idoshamun idoshamun merged commit 8881c3c into main May 5, 2026
13 checks passed
@idoshamun idoshamun deleted the fix/standup-feedback branch May 5, 2026 18:10
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.

1 participant

@idoshamun