Skip to content
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

UI: FECFile Online Dashboard #1908

Closed
18 tasks done
AureliaKhorsand opened this issue May 1, 2024 · 16 comments
Closed
18 tasks done

UI: FECFile Online Dashboard #1908

AureliaKhorsand opened this issue May 1, 2024 · 16 comments
Assignees

Comments

@AureliaKhorsand
Copy link
Contributor

AureliaKhorsand commented May 1, 2024

Business Reason As the UX Principal, I would like to start iterating on the dashboard so it will more closely match the original intend of the page.

Acceptance Criteria

Create placeholders for 3 different sections:

  1. in-progress reports
  2. Cash-on-hand
  3. Recently submitted reports

In-progress –

  • 558w by 404h div
  • border-radius: 4px
  • #E8E8E8
  • Header: "In-progress reports"; 24px; Gandhi-serif bold; #212121
  • Placeholder Card: White; 518x308; with 20px padding left and right
  • Hyperlink: "Start a different report"; points to create report

Cash-on-hand –

  • 558w by 154h div
  • border-radius: 4px
  • #E8E8E8
  • Header: "Cash-on-hand"; 24px; Gandhi-serif bold; #212121
  • Placeholder Card: White; 518x86: with 20px padding left and right

Recently submitted reports –

  • 558w by 578h div
  • border-radius: 4px
  • #E8E8E8
  • Header: "Recently submitted reports"; 24px; Gandhi-serif bold; #212121
  • Placeholder Card: White; 518x482; with 20px padding left and right
  • Hyperlink: "Manage reports"; points to create report

1400xxl

  1. In-progress reports Grey box = w558 x h404 White box = w518
  2. Cash on-hand Grey box = w558 x h154 White box = w518 x h86
  3. Recently submitted reports Grey box = w558 x h578 White box = w518 x 482

1200xl

  1. In-progress reports Grey box = w454 x h404 White box = w414
  2. Cash on-hand Grey box = w454 x h154 White box = w414 x h86
  3. Recently submitted reports Grey box = w454 x h578 White box = w414 x 482

992lg

  1. In-progress reports Grey box = w704 x h404 White box = w664
  2. Cash on-hand Grey box = w704 x h154 White box = w664 x h86
  3. Recently submitted reports Grey box = w704 x h578 White box = w664 x 482

768md

  1. In-progress reports Grey box = w512 x h404 White box = w472
  2. Cash on-hand Grey box = w512 x h154 White box = w472 x h86
  3. Recently submitted reports Grey box = w512 x h578 White box = w472 x 482

576sm

  1. In-progress reports Grey box = w512 x h404 White box = w472
  2. Cash on-hand Grey box = w512 x h154 White box = w472 x h86
  3. Recently submitted reports Grey box = w512 x h578 White box = w472 x 482

QA Notes

  • BA/Design Quality Checklist complete

DEV Notes

the fixed div width inside each breakpoint

  • 1400xxl = 1136px
  • 1200xl = 928px
  • 992lg = 704px
  • 768md = 512px
  • 576sm = 512px

Design

Wireframes
1400xxl

Image

1200xl

Image

992lg

Image

768md

Image

576sm

Image

FECFILE-400

@MitchellTCG
Copy link
Contributor

@JonellaCulmer please review

@JonellaCulmer
Copy link

@MitchellTCG Do we want to reconsider the appearance if we're dropping the gray background elsewhere? I'm wondering if we should also drop the tips for treasurers now until we have a method to populate that section.

@MitchellTCG
Copy link
Contributor

@GreggMoreland please review this ticket and place into ADTQR if it looks good to you

@MitchellTCG MitchellTCG added BA Review and removed UX/UI attention needed UX/UI tasks and/or redesign needed labels May 16, 2024
@MitchellTCG
Copy link
Contributor

@MitchellTCG remove tips for treasurers

@MitchellTCG MitchellTCG added the UX/UI attention needed UX/UI tasks and/or redesign needed label May 22, 2024
@AureliaKhorsand
Copy link
Contributor Author

@MitchellTCG are you able to remove the tips for treasurers so this can be ready for the next sprint?

@MitchellTCG
Copy link
Contributor

@JonellaCulmer
Tips for treasurers removed from mocks

@sasha-dresden
Copy link
Contributor

Development done. Need to code review with @MitchellTCG

@sasha-dresden
Copy link
Contributor

Reviewed with @MitchellTCG and made some minor tweaks for centering the dashboard, font updates and spacing. And made the "Start a different report" open the "Create a Report" dialog on this page, rather than redirecting to the report page. Fixes in and PR created.

@exalate-issue-sync exalate-issue-sync bot removed the UX/UI attention needed UX/UI tasks and/or redesign needed label Jun 17, 2024
@mjtravers
Copy link
Contributor

From CR:

The "Manage reports" link is going to the "Security notifications" page.

Should this ticket be populating the "In-progress Reports" section? If so, some of the report types are not displaying in the list.

Image

Sending ticket back to In Progress.

@sasha-dresden
Copy link
Contributor

Ah, I see the issue. It only happens when you have not chosen to confirm for a year. I changed it from an href to angular's routerLink and it works as expected for that situation now.

As for the "In-progress Reports" section that's more of a work in progress. There will be a later ticket for building that out which will provide how that will function fully. All I wanted to do was make sure that when there are multiple reports the white boxes that will eventually be filled with data, are arranged properly. The screenshots showed 2 but the AC mentioned the total area, so I built this out as a placeholder till we get the official version.

@mjtravers
Copy link
Contributor

@sasha-dresden Thanks for the clarification about the wireframes. That make sense.

@mjtravers
Copy link
Contributor

Passes CR. Sending to QA.

@WiseQA
Copy link

WiseQA commented Jun 19, 2024

@MitchellTCG @AureliaKhorsand the Acceptance Criteria states 4 different sections. Verifying that there should only be 3 and not 4 sections. Thanks

Create placeholders for 4 different sections:

  1. in-progress reports
  2. Cash-on-hand
  3. Recently submitted reports

@WiseQA
Copy link

WiseQA commented Jun 20, 2024

@MitchellTCG verified states should be changed from 4 different sections to 3 different sections. Per our review and discussion updated ticket as such. Thanks !

@WiseQA
Copy link

WiseQA commented Jun 20, 2024

QA review verified FECFile Online Dashboard updated with the current UX / Functionality only documented in this ticket.

Verified the following section placeholder's have been added to the dashboard.

  1. in-progress reports
  2. Cash-on-hand
  3. Recently submitted reports

Image


Select the "Start a different report >" link in the "In-progress reports" section.


Image


Verify "Create a new report" popup window is displayed and selecting a "Form Type" functionality is working.

Image


Select the "Manage reports >" link in the "Recently submitted reports" section.

Image


Verify "Manage reports" page is displayed.

Image


QA Review Completed. Moved to Stage Ready.

@WiseQA WiseQA assigned sasha-dresden and unassigned WiseQA Jun 20, 2024
Copy link

akhorsand commented: Accepted during PI Planning Sprint Review on 7/8/2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants