GSA / notifications-admin

The UI of Notify.gov
https://notify.gov
Other
10 stars 2 forks source link

made changes/fixes to tour flow #1629

Closed heyitsmebev closed 2 days ago

heyitsmebev commented 2 weeks ago

This PR focuses on cleaning up issues/bugs in the tour flow. It targets styling, backlink, and template layout

Ticket: https://github.com/GSA/notifications-admin/issues/1256

The styling was fixed when the new withnav_template.html was merged. However, the tour flow was a bit odd. When the user went through the tour flow, it would ask the user to select a time when there were no input fields, and the headers were not correctly describing the current step in the tour flow. In addition, the back link was not properly linking to the correct area. I fixed the tour flow by isolating our send and tour flow so that the tour flow would look more complete and unified. Originally, it was inheriting the changes made to the send flow.

The old verison

image

New Version

image