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

Profile Form doesnt close after submitting #117

Open
ruddi10 opened this issue Mar 8, 2021 · 14 comments
Open

Profile Form doesnt close after submitting #117

ruddi10 opened this issue Mar 8, 2021 · 14 comments
Labels
Category: Coding Changes to code base or refactored code that doesn't fix a bug. Category: User Interface Improvements or additions to design. Hacktoberfest Status: Available Issue was approved and available to claim or abandoned for over 3 days.

Comments

@ruddi10
Copy link

ruddi10 commented Mar 8, 2021

Describe the bug

When a user edit or fill the profile even after pressing the submit button the form remains open and I think that creates confusion I would rather suggest that we close the form .

To Reproduce

Steps to reproduce the behavior:

  1. Go to dashboard
  2. Click on Edit Profile
  3. Change some information and click Submit button
  4. The form will not close

Expected behavior

I think we should close the form and moreover can use a tost notification once the profile is updated.

Screenshots

Screenshot from 2021-03-08 17-43-44

Expected-
Screenshot from 2021-03-08 17-44-38

@ruddi10
Copy link
Author

ruddi10 commented Mar 8, 2021

If this issue is fine. Kindly assign it to me @codesankalp @keshakaneria

@codesankalp codesankalp added Category: Coding Changes to code base or refactored code that doesn't fix a bug. Category: User Interface Improvements or additions to design. labels Mar 8, 2021
@codesankalp
Copy link
Member

Sure @ruddi10. I am assigning this to you.
Also, one suggestion, If after submission backends respond with an error then don't close the form and display the error inside a message box. Also, don't forget to write a test for this feature.

@ruddi10
Copy link
Author

ruddi10 commented Mar 8, 2021

Also should I show a pop-up incase the update is successful or not? @codesankalp

@keshakaneria
Copy link
Member

Also should I show a pop-up incase the update is successful or not? @codesankalp

Pop up or a line in red on top above Dashboard text will be fine to let the user know if the form is submitted.

@codesankalp
Copy link
Member

Also should I show a pop-up incase the update is successful or not? @codesankalp

Inline Error message box which disappears after some time.
Use Semantic UI.

@ruddi10
Copy link
Author

ruddi10 commented Mar 8, 2021

I was asking incase its successful

@keshakaneria
Copy link
Member

Also should I show a pop-up incase the update is successful or not? @codesankalp

Pop up or a line in red on top above Dashboard text will be fine to let the user know if the form is submitted.

You can go for this @ruddi10. If you have any queries do let us know.

@codesankalp
Copy link
Member

I was asking incase its successful

Same success message box provided in Semantic UI.
And as mentioned by @keshakaneria you can go with your own idea and we will review it in PR.

@ruddi10
Copy link
Author

ruddi10 commented Mar 9, 2021

@keshakaneria @codesankalp I have fixed this issue just wanted to know the code formatter used for this repo as I use Prettier and its changing the formatting on save so it might be a problem

@codesankalp
Copy link
Member

This will solve after merging of #94.
You can make PR with formatted code.

@ruddi10
Copy link
Author

ruddi10 commented Mar 9, 2021

ok thanks

@codesankalp codesankalp added Status: Available Issue was approved and available to claim or abandoned for over 3 days. and removed Status: Available Issue was approved and available to claim or abandoned for over 3 days. labels Apr 12, 2021
@isabelcosta isabelcosta added Hacktoberfest Status: Available Issue was approved and available to claim or abandoned for over 3 days. labels Oct 2, 2021
@eleensmathew
Copy link

Can I work on this? Can you please guide me since I am new

@jivteshsingh
Copy link

If the issue is still there, kindly assign it to me.

@Bishwajitpvt
Copy link

If this issue is still active, I can work on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Coding Changes to code base or refactored code that doesn't fix a bug. Category: User Interface Improvements or additions to design. Hacktoberfest Status: Available Issue was approved and available to claim or abandoned for over 3 days.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants