Skip to content

Realign MessageActionBar to Figma designs#32722

Merged
t3chguy merged 8 commits intodevelopfrom
t3chguy/mab-design-tweaks
Mar 9, 2026
Merged

Realign MessageActionBar to Figma designs#32722
t3chguy merged 8 commits intodevelopfrom
t3chguy/mab-design-tweaks

Conversation

@t3chguy
Copy link
Copy Markdown
Member

@t3chguy t3chguy commented Mar 4, 2026

@gaelledel
Copy link
Copy Markdown

Screenshot 2026-03-04 at 17 29 03

Seems we don't have the same amount of padding above and on the left of the emoji?
I also wonder If it would be possible to make the tooltips appear above the component (might be another line of work though)

t3chguy added 2 commits March 4, 2026 23:37
Signed-off-by: Michael Telatynski <[email protected]>
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Copy link
Copy Markdown
Member Author

t3chguy commented Mar 4, 2026

@gaelledel done & done

@t3chguy t3chguy marked this pull request as ready for review March 4, 2026 23:40
@t3chguy t3chguy requested a review from a team as a code owner March 4, 2026 23:40
t3chguy added 2 commits March 4, 2026 23:48
Signed-off-by: Michael Telatynski <[email protected]>
Signed-off-by: Michael Telatynski <[email protected]>
Copy link
Copy Markdown

@gaelledel gaelledel left a comment

Choose a reason for hiding this comment

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

Perfetto thank you

Copy link
Copy Markdown
Member

@florianduros florianduros left a comment

Choose a reason for hiding this comment

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

  • alignment
Image

The message action bar is overflowing over the top of the message where on develop it doesn't (or 1 or 2px?):
Image.

  • gap between each button
    Is it possible to only rely on the gap properties of the MessageActionBar?. The current gap is the addition of gap + 2 * --MessageActionBar-size-margin.

Signed-off-by: Michael Telatynski <[email protected]>
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy t3chguy enabled auto-merge March 6, 2026 15:43
@t3chguy t3chguy added this pull request to the merge queue Mar 9, 2026
Merged via the queue into develop with commit 095b407 Mar 9, 2026
44 checks passed
@t3chguy t3chguy deleted the t3chguy/mab-design-tweaks branch March 9, 2026 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants