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

Theme: Fix placeholder image references #2448

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

EricDunsworth
Copy link
Member

@EricDunsworth EricDunsworth commented Nov 25, 2024

Some of GCWeb's code samples from recent years were referencing placeholder images from placeholder.com and placehold.it. But those sites underwent changes in ownership in March 2023 and consequently ceased serving placeholder images.

This deals with it by replacing all references to via.placeholder.com and placehold.it with dummyimage.com (and swaps ? characters with & where custom text is specified to fit the latter's URL writing scheme).

GCWeb has historically used dummyimage.com and that site still exists to this day.

Left the following references to placeholder.com "as-is" in the following WCAG assessments since they're meant to represent specific points in time:

Some of GCWeb's code samples from recent years were referencing placeholder images from placeholder.com and placehold.it. But those sites underwent changes in ownership in March 2023 and consequently ceased serving placeholder images.

This deals with it by replacing all references to via.placeholder.com and placehold.it with dummyimage.com (and swaps ? characters with & where custom text is specified to fit the latter's URL writing scheme).

GCWeb has historically used dummyimage.com and that site still exists to this day.

Left the following references to placeholder.com "as-is" in the following WCAG assessments since they're meant to represent specific points in time:
* templates/institutional-landing/reports/a11y-1.json
* templates/institutional-landing/reports/a11y-2.json
@Garneauma
Copy link
Contributor

Pre-approved upon successful review. This is only a content change.

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.

2 participants