-
Notifications
You must be signed in to change notification settings - Fork 3
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
[394] Update Submissions Stats Dashboards #434
Conversation
'green-cool-vivid-60v': #216E1F, | ||
'orange-warm-vivid-50v': #CF4900, | ||
'red-vivid-60v': #B50909 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌈
@@ -17,6 +17,9 @@ def index | |||
@unassigned_submissions = @evaluator_assignments. | |||
where(status: %i[unassigned recused_unassigned]). | |||
ordered_by_status | |||
@recused_submissions = @evaluator_assignments. | |||
where(status: %i[recused]). | |||
ordered_by_status |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do these need to be ordered_by_status?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, you're right - recused submissions don't need to be ordered by status 😅
<div class="display-flex flex-row tablet:margin-left-auto tablet:margin-top-0 margin-top-3"> | ||
<div class="text-center bg-green-cool-vivid-60v padding-1 margin-right-1" style="width: 32%"> | ||
<span class="font-sans-xl text-white text-bold display-block"><%= @submissions_count["completed"] || 0 %></span> | ||
<span class="font-sans-3xs tablet:font-sans-sm text-white">COMPLETED<br>EVALUATIONS</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
capitalization can be handled by css property text-transform
, which is text-uppercase
in USWDS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e.g.
<span class="font-sans-3xs tablet:font-sans-sm text-white">COMPLETED<br>EVALUATIONS</span> | |
<span class="font-sans-3xs tablet:font-sans-sm text-white text-uppercase">Completed<br>Evaluations</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just some minor non-blocking touchup, otherwise good to go 👍
Linked ticket: #394
Update submission stats dashboards:
Add colors required in the updated figma.
Evaluator Submissions list
Challenge Managers Submission list
Evaluator Management Submissions list page
WAVE