Skip to content
This repository was archived by the owner on Jun 7, 2020. It is now read-only.

[CHORE] Bouncing reactors & badge all the time for unreads#2051

Merged
filipealva merged 5 commits intodevelopfrom
chore/scrolling_reactions
Jul 20, 2018
Merged

[CHORE] Bouncing reactors & badge all the time for unreads#2051
filipealva merged 5 commits intodevelopfrom
chore/scrolling_reactions

Conversation

@rafaelks
Copy link
Copy Markdown
Contributor

@RocketChat/ios

@filipealva
Copy link
Copy Markdown
Contributor

A little description on the PR is good when there's no issue associated 🤓

About the bouncing: awesome!

About the unread badge for all unread messages: I like the idea but the ! is so thin that it's annoying to look at, and I would easily confuse it with the number 1.

I tried letting the font bold when showing the ! it looked a little bit better for me. Maybe we should proceed with the ! bold for now and then work with Thiago to improve it. What do you think about?

@filipealva
Copy link
Copy Markdown
Contributor

screen shot 2018-07-20 at 15 46 21

Just one more thing 🤓 we could increase the spacing between the label and the badge just a little bit.

Copy link
Copy Markdown
Contributor

@filipealva filipealva left a comment

Choose a reason for hiding this comment

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

I'm approving it since the spacing issue is not related to this PR it's just a suggestion

@filipealva filipealva merged commit 0963d7e into develop Jul 20, 2018
@rafaelks rafaelks deleted the chore/scrolling_reactions branch July 25, 2018 12:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants