You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Go back to previous modal when clicked, instead of Close #147
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
Integrate Go Back option instead of Close submit button in PR Actions modal.
@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 ?
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 /github Username/RepositoryName pulls is executed. I am approaching in this way because I have seen that there is not so much deep nesting of modals, we have upto level 2, meaning suppose when clicking on view file action button the view file modal will show up and next there is no modal.
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.
Activity
umangutkarsh commentedon Apr 21, 2024
Hi @samad-yar-khan . What do you think about this? Please let me know. Thanks.
samad-yar-khan commentedon May 5, 2024
@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 ?
umangutkarsh commentedon May 7, 2024
Yes sure @samad-yar-khan . Is it okay if I give you updates after I make some progress?
Thanks.
samad-yar-khan commentedon May 7, 2024
yes, please feel free to join the channel : https://open.rocket.chat/channel/GitHub-App-Community-Developers
devanshkansagra commentedon Dec 25, 2024
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?
devanshkansagra commentedon Dec 26, 2024
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
/github Username/RepositoryName pullsis executed. I am approaching in this way because I have seen that there is not so much deep nesting of modals, we have upto level 2, meaning suppose when clicking onview fileaction button the view file modal will show up and next there is no modal.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?