Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Improve style of toasts to match Figma #4613

Merged
merged 5 commits into from
May 26, 2020
Merged

Improve style of toasts to match Figma #4613

merged 5 commits into from
May 26, 2020

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented May 21, 2020

Fixes element-hq/element-web#11914
Fixes element-hq/element-web#11923
Subset of #4609

icon-less variant for future use
image

image
image

  • min-width of buttons
  • update style of the toast counter to be right-aligned, muted and smaller
  • disable text decoration of links inside the toasts
  • fix background colour, stacking background and the drop shadow
  • fixes padding to match Figma

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy requested review from a team May 21, 2020 15:33
t3chguy added 4 commits May 21, 2020 16:52
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Copy link
Contributor

@nadonomy nadonomy left a comment

Choose a reason for hiding this comment

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

LGTM, with a note to myself & @benedikrok that we need to iterate on the iconography and spit polish the sizing/spacing a bit in future, perhaps as a part of the upcoming notifications work.

@t3chguy t3chguy merged commit c00fd32 into develop May 26, 2020
@t3chguy t3chguy deleted the t3chguy/toasts1 branch May 26, 2020 11:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stacked toasts on dark theme have an awkward light theme underlay Toasts in dark theme are hard to see
3 participants