Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMPROVE] Added a Reset Button in the Account Profile Page #24078

Merged
merged 5 commits into from
Jan 7, 2022

Conversation

aswiniip
Copy link
Contributor

@aswiniip aswiniip commented Jan 1, 2022

Proposed changes (including videos or screenshots)

Screencast-from-02-01-22-01_12_40-AM-IST.mp4

Issue(s)

Steps to test or reproduce

Further comments

I added a reset button which resets all the changes made while editing the profile. It seemed like a necessary improvement because whenever i had to not save the changes, i had to go to a different page and come back

@murtaza98
Copy link
Contributor

Hey, @aswinidev This indeed seems like a good improvement. I wanted to suggest a couple of improvements here

  1. Make the reset button theme different from the Save button (a positive action)
  2. The reset button should only be visible (or clickable) when the user has changed something which isn't yet saved. Look into the Cancel button on the Admin settings panel for inspiration.
    image

@aswiniip
Copy link
Contributor Author

aswiniip commented Jan 5, 2022

Sorry for the late reply. Must have missed my mails 😥 . I made the changes. @murtaza98 please review!

Now it looks and works like this:

Screencast-from-05-01-22-02_34_40-PM-IST.mp4

@murtaza98 murtaza98 self-requested a review January 7, 2022 06:47
@murtaza98 murtaza98 merged commit eb7e37d into RocketChat:develop Jan 7, 2022
gabriellsh added a commit that referenced this pull request Jan 17, 2022
…ove/setup-wizard

* 'develop' of github.com:RocketChat/Rocket.Chat: (176 commits)
  [IMPROVE] Admin page header buttons consistency (#24168)
  i18n: Language update from LingoHub 🤖 on 2022-01-17Z (#24193)
  [FIX] Integration section crashing opening in My Account (#24068)
  [IMPROVE] Rewrite roomNotFound to React Component (#24044)
  Regression: Enable custom emoji on admin custom status page (#24186)
  Chore: Update Meteor to 2.5.3 (#24075)
  [NEW] Apple Login (#24060)
  Chore: Update Apps-Engine to 1.29.2 (#24171)
  feat: enabling emoji on custom status (#24170)
  [FIX] App Framework Enable hanging indefinitely (#24158)
  [FIX] CSV Importer failing to import users (#24090)
  Fix Engagement Dashboard API requests (#24142)
  Language update from LingoHub 🤖 (#24127)
  Chore: Migrate useOutsideClick to fuselage-hooks (#24133)
  Revert "Use fibers to store context"
  Use fibers to store context
  Chore: Include REG_TOKEN in docker-compose (#24123)
  [FIX] Custom Emoji Image preview #24117
  [IMPROVE] Added a Reset Button in the Account Profile Page (#24078)
  Revert: "[IMPROVE] Throw 404 error in invalid endpoints" (#24118)
  ...
@sampaiodiego sampaiodiego mentioned this pull request Jan 29, 2022
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