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

Update how 'Overview' link opens for Civic Tech Index #2237

Closed
3 tasks done
Tracked by #2211
macho-catt opened this issue Sep 8, 2021 · 2 comments · Fixed by #2507
Closed
3 tasks done
Tracked by #2211

Update how 'Overview' link opens for Civic Tech Index #2237

macho-catt opened this issue Sep 8, 2021 · 2 comments · Fixed by #2507
Assignees
Labels
Feature: Refactor HTML good first issue Good for newcomers role: front end Tasks for front end developers size: missing Status: Updated No blockers and update is ready for review time sensitive Needs to be worked on by a particular timeframe

Comments

@macho-catt
Copy link
Member

macho-catt commented Sep 8, 2021

Overview

As a user, clicking on the 'Overview' link for the Civic Tech Index project should open a new tab to the pdf file directly. This is made as an action item of #2211

Details

When clicking the 'Overview' link, each project page acts differently. Some open to a google docs page, others open to a github page that hosts the pdf file, and another downloads the pdf file locally to the machine. There needs to be a standard in how the pdf files open. Developers decided during the backend meeting that pdfs should be opened as a separate tab to the file directly, since modern browsers have the ability to view pdfs.

Example:
How Civic Tech Index's pdf file currently opens
How it should open

Action Items

  • Download a copy of Civic Tech Index's overview pdf and put it in /assets/pdfs
  • Change the 'Overview' link for the markdown file to point to the pdf file living under /assets/pdfs
  • Ensure that the links in both the project page and the project's dedicated page open the pdf file properly after the change

Resources/Instructions

Markdown file
Project Page
Location for pdf files
Jekyll Static Files

@github-actions github-actions bot added Feature Missing This label means that the issue needs to be linked to a precise feature label. role missing size: missing labels Sep 8, 2021
@macho-catt macho-catt changed the title Change the 'Overview' link for Civic Tech Index project to open as a pdf file in the browser instead of opening a github page that hosts the pdf file Update how 'Overview' link opens for Civic Tech Index Sep 8, 2021
@macho-catt macho-catt added Feature: Refactor HTML Ready for Prioritization role: front end Tasks for front end developers Size: Good second issue and removed Feature Missing This label means that the issue needs to be linked to a precise feature label. role missing size: missing labels Sep 8, 2021
@ExperimentsInHonesty ExperimentsInHonesty added the time sensitive Needs to be worked on by a particular timeframe label Nov 9, 2021
@JimGeist
Copy link
Member

Only blocker right now is lack of time! I hope to have a change to look at this over the weekend.

@JimGeist
Copy link
Member

Completed issue and issued pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: Refactor HTML good first issue Good for newcomers role: front end Tasks for front end developers size: missing Status: Updated No blockers and update is ready for review time sensitive Needs to be worked on by a particular timeframe
Projects
Development

Successfully merging a pull request may close this issue.

4 participants