-
Notifications
You must be signed in to change notification settings - Fork 39
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
Go back to previous modal when clicked, instead of Close #147
Comments
Hi @samad-yar-khan . What do you think about this? Please let me know. Thanks. |
@umangutkarsh makes sense. I believe this was earlier inbuilt in RocketChat and they then removed this stacking. I think stacking its like an application UI would make for a better UX. Would you like to work on this ? |
Yes sure @samad-yar-khan . Is it okay if I give you updates after I make some progress? Thanks. |
yes, please feel free to join the channel : https://open.rocket.chat/channel/GitHub-App-Community-Developers |
Hey @samad-yar-khan and @umangutkarsh, this issue has been idle for a while. If it's okay with both of you, can I work on this issue/feature? |
Hey @samad-yar-khan and @umangutkarsh, a suggestion, can we add a normal action button go-back at the end? And in that on clicking it, it will display the modal which was displayed when There may still be some minor inconsistencies in the ux, such as the need to scroll all the way down to navigate back. However, this small inconsistency is not as significant as having to execute the entire command and re-open the modal again. So what are your thoughts on this approach? |
Why do we need this ?
To make the user experience better.
Currently, when this command ->
/github Username/RepositoryName pulls
is entered, action buttons along with link to PR are rendered. And in the PR actions modal when the desired buttons are clicked, for example - View File, View Changes, Comments, etc, the desired modal is opened up but then there is no option to go back, instead the user has to close the modal and again re-open it to select other option in the PR actions modal.As a GitHub App developer
I need to: Implement a feature to go back to previous modal (PR actions modal) instead of closing it.
So That: The user doesn't have to re-open the modal again and again which doesn't provide good user experience.
Acceptance Criteria
Go Back
option instead ofClose
submit button in PR Actions modal.Screenshots/Record
gh-ux.webm
The text was updated successfully, but these errors were encountered: