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

Updates to the Advocacy Page and PDFs #38523

Merged
merged 12 commits into from
Feb 3, 2021
Merged

Conversation

JillianK
Copy link
Contributor

All of the changes are outlined in the spec

Here is an example updated state facts PDF to be compared to the old state facts PDF

New advocacy page for national stats:
image

Old advocacy page for national stats:
Screen Shot 2021-01-12 at 10 37 25 AM

New advocacy page for state stats:
image

Old advocacy page for state stats:
Screen Shot 2021-01-12 at 10 37 57 AM

Links

Testing story

Reviewer Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

@JillianK JillianK requested a review from a team January 12, 2021 17:38
@@ -13,6 +13,7 @@ state-facts: true
%script{src:'https://www.google.com/jsapi'}
%link{:rel=>'stylesheet', :type=>'text/css', :href=>'/css/state-facts-print.css'}
%link{:rel=>'stylesheet', :type=>'text/css', :href=>'/css/state-facts.css'}
%link{:rel=>'stylesheet', :href=>'https://maxcdn.bootstrapcdn.com/font-awesome/4.6.3/css/font-awesome.min.css'}
Copy link
Member

Choose a reason for hiding this comment

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

Pegasus should already have its own copy of this file...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For some reason, without this line the font awesome icons disappear when we generated the PDF

Copy link
Member

Choose a reason for hiding this comment

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

Yes, it probably makes sense to add font awesome, but I just think we should refer to our own served version, like we do here: https://github.com/code-dot-org/code-dot-org/blob/staging/pegasus/sites/code.org/views/theme.haml#L7

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ohhh that makes sense, thanks for pointing that out!

@@ -479,7 +396,7 @@ state-facts: true
%div.policy-item
- if state_info[:state_plan_s] == 'Yes'
%div
<i class="far fa-check-square"></i>
Copy link
Member

Choose a reason for hiding this comment

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

Did these not work before?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, the checked squares are showing the invalid icon symbol right now
Screen Shot 2021-01-13 at 3 56 17 PM

@@ -35,6 +35,9 @@
%strong= format_integer_with_commas(jobs_nationwide)
Open computing jobs nationwide

%strong=number_to_percentage(47, precision: 0)
Copy link
Member

Choose a reason for hiding this comment

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

Ah, do we plan to update this number somewhat regularly?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We plan to have engineering update this number every Fall

@JillianK JillianK merged commit b23d32e into staging Feb 3, 2021
@JillianK JillianK deleted the jk-advocacy-page-pdfs-update branch February 3, 2021 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants