Added new button to save request modal

[shipko]

Description

I noticed that you can't cancel modal window when saving a request.

• Not Completed
• Completed

I suggest adding a third "cancel" button, which will close the window without performing an action.

Checks

• My pull request adheres to the code style of this project
• My code requires changes to the documentation
• I have updated the documentation as required
• All the tests have passed

[JoelJacobStephen]

An extra cancel button is not required. Let the cross icon on the top right of the modal handle the cancellation case. So you can remove the entire span container

[shipko]

Thanks for review!
Maybe you're right. I thought the user might have doubts about the action of the upper right cross. Adding a new button will add clarity. If your position remains unchanged, then I will correct it

[JoelJacobStephen]

@shipko The cross button inherently means to cancel the action. The same design and implementation is followed for most modals in the app. So we will follow the same. Let's remove the cancel button as it's redundant.

[shipko]

@JoelJacobStephen done!

[JoelJacobStephen]

@shipko After having an internal discussion regarding this PR, we have decided to go with the following approach:

1. Yes Button - Opens the save dialog modal and closes the tab
2. No Button - Doesn't save and closes the tab
3. Cancel Icon - (Present at the top right of the modal) - Neither saves nor closes the tab

Based on the above approach, I have suggested few changes to your implementation. Please go through the review comments and then we can go ahead with this PR.

[JoelJacobStephen]

LGTM 💯

[nivedin]

Just this minor addition, LGTM 💯