fecgov / fecfile-web-app

Other
7 stars 4 forks source link

Add Accordion Side Navigation #346

Closed MitchellTCG closed 2 years ago

MitchellTCG commented 2 years ago

Purpose

Create the Report Workflow Accordion Side Navigation.

Business Reason

As a Committee Administrator, I will be able to navigate thru the Form3X report workflow. I will be able to move back and forth through the workflow

Dev Notes

This is base on the FEC Pattern library component https://fec-pattern-library.app.cloud.gov/components/detail/side-navigation.html The blue background is a gradient that goes from #1A4F82 to #112E51

Building Specs

Accordion Navigation

Note: Clicking the top level of the "accordion" menu expands the menu. Clicking on another top level item will collapse any others. Users are not taken to a page until the select an option within the menu itself.

Section 1: ENTER A TRANSACTION - [ALL CAPS]

Section 2: REVIEW A REPORT

Section 3: SUBMIT YOUR REPORT

NAV – 1.png

QA Notes

This a visual check. Links will be add in next Sprint. The report accordion menu shows up on all site pages for this ticket and the info about the report above the accordion menu is place holder information. The display of the accordion menu and the proper display of the report info will be handled in the follow up ticket, #350.

DEV Notes

null

Design

null

MitchellTCG commented 2 years ago

I can't add all the links that exist until Dev and Stage are back on.

MitchellTCG commented 2 years ago

@AureliaKhorsand

Updated

AureliaKhorsand commented 2 years ago

@MitchellTCG updated requirement from PI3 Planning - the Sign and Submit step will actually be on the same page, and will be the same action for the user. I suspect we want to combine these in the menu. Otherwise, this looks good and is ready for ADTR.

MitchellTCG commented 2 years ago

@AureliaKhorsand Re: the Sign & Submit section — Much like the Create a Report section, the purpose of the Nav is help assist users in finding what they need like a Table of Contents or Index. Multiple links can go to the same page. If the pages are long, we can add anchor links if needed.

Not a blocker.

AureliaKhorsand commented 2 years ago

6/23 Refinement Meeting: Strip out persistent report information from wireframes and separate into another ticket. Update wording to indicate that this is just a visual check for the accordion menus.

AureliaKhorsand commented 2 years ago

Design meeting 6/23: Want to move Create a Report information to pop up/single page before entering the report itself. Also will move Add report level memo to another accordion.

Note that #166 will need an updated wireframe.

MitchellTCG commented 2 years ago

@AureliaKhorsand updated

AureliaKhorsand commented 2 years ago

@MitchellTCG what is the decision on the "top level" for the accordions? Are they links to pages or just expanders? Also, didn't we discuss removing "Submission Status" unless it is a closed report? Fine if we keep it, we'll just need to add #292 as a blocked.

mjtravers commented 2 years ago

Passes CR, sending to QA. See additional QA notes added to ticket description.

WiseQA commented 2 years ago

QA review as a visual navigation side bar has been created according to wireframes above. No links are functional at this point in time.

image.png

image.png

image.png

QA review completed on visual inspection. Moved to Stage Ready.