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

Tbolt/4506 Updates to proposed budget and executive summary #4583

Merged
merged 38 commits into from
Mar 21, 2023

Conversation

tbolt
Copy link
Contributor

@tbolt tbolt commented Mar 2, 2023

Resolves #4506

Description

Updates to table designs

Chromatic approval

Significant changes or possible side effects

  • Styling changes for many of the table classes. Possible side-effects: Updating a CSS class has a cascading effect and updates an unintended component/table

Automated test cases written

No new tests needed for these design changes. Tests were updated where needed.

Steps to manually verify this change

  1. Login to view APDs
  2. View an APD with MMIS type
  3. Verify Changes are according to ticket + Figma
  4. Repeat with an HITECH APD type
  5. Since we changed the order of the tables - swapping the federal and state share columns - test that state share data and federal share data is displaying in their correct columns

This pull request is ready to code review when

  • Automated tests are updated (and all tests are passing)
  • New automated test cases are documented above
  • Pull request has been labeled, if applicable with feature, content, bug,
    tests, refactor
  • Associated OpenAPI documentation has been updated
  • The experience passes a basic manual accessibility audit (keyboard nav,
    screenreader, text scaling) OR an exemption is documented

This pull request is ready to test when

  • Code has been reviewed by someone other than the original author

This pull request is ready to review when the QA has

  • Verified the functionality related to the change
  • Verified that the change works with Narrator on Windows
  • Verified that the change works with VoiceOver on Mac
  • Verified all updated pages with the WAVE tool
  • Verified tab and keyboard navigation functionality

This pull request can be merged when

  • Design has approved the experience
  • Product has approved the experience

@codecov
Copy link

codecov bot commented Mar 2, 2023

Codecov Report

Merging #4583 (6956342) into main (57ec9df) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4583      +/-   ##
==========================================
+ Coverage   94.30%   94.31%   +0.01%     
==========================================
  Files         273      273              
  Lines        8660     8694      +34     
  Branches     1721     1746      +25     
==========================================
+ Hits         8167     8200      +33     
- Misses        469      470       +1     
  Partials       24       24              
Flag Coverage Δ
api ∅ <ø> (∅)
common 100.00% <ø> (ø)
web 94.05% <100.00%> (+0.01%) ⬆️
Impacted Files Coverage Δ
e2e/cypress/page-objects/activity-schedule-page.js 100.00% <ø> (ø)
e2e/cypress/page-objects/export-page.js 64.95% <ø> (ø)
...vities/cost-allocation/CostAllocateFFPQuarterly.js 100.00% <ø> (ø)
...eb/src/pages/apd/activities/ffp/CostAllocateFFP.js 97.61% <ø> (ø)
...b/src/pages/apd/activities/ffp/FedStateSelector.js 100.00% <ø> (ø)
.../src/pages/apd/activities/ffp/MatchRateSelector.js 100.00% <ø> (ø)
...rc/pages/apd/executive-summary/ExecutiveSummary.js 96.96% <ø> (ø)
...src/pages/apd/proposed-budget/IncentivePayments.js 94.91% <ø> (ø)
...es/apd/proposed-budget/SummaryActivityBreakdown.js 100.00% <ø> (ø)
...es/apd/proposed-budget/SummaryKeyStatePersonnel.js 100.00% <ø> (ø)
... and 11 more

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 57ec9df...6956342. Read the comment docs.

@cms-eapd-bot
Copy link

cms-eapd-bot commented Mar 2, 2023

See this pull request in action: https://ec2-34-193-194-48.compute-1.amazonaws.com

6956342

Copy link
Contributor

@thetif thetif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@thetif thetif requested review from Sun-Mountain and amyd11 March 2, 2023 20:54
@tbolt tbolt marked this pull request as draft March 4, 2023 12:29
@tbolt tbolt added content Some design issues are more specifically content issues feature PR label for release log labels Mar 7, 2023
Copy link
Contributor

@amyd11 amyd11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job!

@tbolt tbolt requested review from mirano-darren and removed request for Sun-Mountain March 15, 2023 13:44
@mirano-darren mirano-darren self-assigned this Mar 15, 2023
@mirano-darren
Copy link
Contributor

Looks good to me! Note: Executive summary couldn't be checked (page still crashes) so I checked them in Chromatic

Given When Then Covered
a HITECH or MMIS APD On the Proposed Budget page, Executive Summary page, Budget and FFP page, and the Activity Schedule Summary page All changes to tables should only affect the Activity total cost table, Combined Activity Cost table, State and Contractor Cost breakdown table, Summary Budget Table, Budget and FFP table, Executive Summary Table, Schedule Summary by Activity, and Milestone Tables by Activity
a HITECH or MMIS APD On the Proposed Budget page, Executive Summary page, Budget and FFP page, and the Activity Schedule Summary page If a table has more than 4 rows, each row should alternate in color. Even numbered rows should be gray (#F1F1F1)
a HITECH or MMIS APD Tables on the Proposed Budget page and Budget and FFP page Subtotal rows should have a light yellow background color and Grand total rows should have a dark yellow background color
a HITECH or MMIS APD Tables on the Executive Summary page Total row should be light yellow. This is an exception to the totals/subtotals color scheme. (Verified w/ Akua)
a HITECH or MMIS APD On the Proposed Budget page Help text for the Summary Budget table section should be updated
a HITECH or MMIS APD Tables on the Activity Schedule Summary; Combined Activity Costs and State and Contractor Cost Breakdown tables on the Proposed budget The table header background color should change from black to gray to match the other tables
a HITECH or MMIS APD Summary Budget Tables on the Proposed Budget Federal total should come first before State total
a HITECH or MMIS APD Milestone Tables by Activity on the Activity Schedule Summary page Table header should be changed to the activity name. Subheaders should be Milestone List and Target completion date

Copy link
Contributor

@jeromeleecms jeromeleecms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice - this looks like the Figma. Still other tickets to revise the tables, but this covers the scope of the ticket!

Copy link

@SGilliamA1M SGilliamA1M left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work on the table updates @tbolt!

Copy link
Contributor

@mirano-darren mirano-darren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, Well done!

@tbolt tbolt merged commit 475c413 into main Mar 21, 2023
@tbolt tbolt deleted the tbolt/4506-proposed-budget-executive-summary-updates branch March 21, 2023 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Some design issues are more specifically content issues feature PR label for release log
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Updates to Proposed Budget and Executive Summary
8 participants