Skip to content

Conversation

@sushant52
Copy link
Contributor

@sushant52 sushant52 commented Jan 6, 2021

Proposed changes (including videos or screenshots)

Made user avatar change buttons to be descriptive of what they do.

Issue(s)

Closes Issue #20082

Initially

gif_without_title

Finally

gif_with_title

@sushant52
Copy link
Contributor Author

@gabriellsh Done. Have a look!

@ggazzo ggazzo changed the title Title for user avatar change buttons [IMPROVE] Title for user avatar buttons Jan 12, 2021
@ggazzo ggazzo merged commit b9f2f35 into RocketChat:develop Jan 12, 2021
gabriellsh added a commit that referenced this pull request Jan 14, 2021
…rewrite-announcement

* 'develop' of github.com:RocketChat/Rocket.Chat: (23 commits)
  Regression: Info Page Icon style and usage graph breaking (#20180)
  Chore: Change console.warning() to console.warn() (#20200)
  [FIX] Room's list showing all rooms with same name (#20176)
  Regression: Change sort icon (#20177)
  [FIX] Wrong userId when open own user profile (#20181)
  [FIX] Livechat.RegisterGuest method removing unset fields (#20124)
  [IMPROVE] Rewrite Prune Messages as React component (#19900)
  [FIX] Change header's favorite icon to filled star (#20174)
  [FIX] Initial values update on Account Preferences  (#19938)
  [FIX] Unable to reset password by Email if upper case character is present (#19643)
  [FIX] Video call message not translated (#18722)
  [NEW] Server Info page (#19517)
  [IMPROVE] Title for user avatar buttons (#20083)
  [FIX] Admin User Info email verified status (#20110)
  [IMPROVE] Tooltip added for Kebab menu on chat header (#20116)
  [FIX] Translate keyword for 'Showing results of' in tables (#20134)
  [FIX] Markdown added for Header Room topic (#20021)
  [FIX] Status circle in profile section (#20016)
  [FIX] Normalize messages for users in endpoint chat.getStarredMessages (#19962)
  [FIX] minWidth in FileIcon to prevent layout to broke (#19942)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Feb 1, 2021
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.

3 participants