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

Stopgap fix for broken feed display images #224

Merged
merged 2 commits into from
Jun 8, 2023
Merged

Conversation

howard-e
Copy link
Contributor

@howard-e howard-e commented May 31, 2023

Temporary fix for broken feed display example images, until further research is done into packaging the assets together for the Jekyll build system. Inspired by w3c/aria-practices#2705

Related issues:

Link to deploy preview's feed-display example page

@netlify
Copy link

netlify bot commented May 31, 2023

Deploy Preview for aria-practices ready!

Name Link
🔨 Latest commit ce46b1a
🔍 Latest deploy log https://app.netlify.com/sites/aria-practices/deploys/647f79831f70b1000819ddb3
😎 Deploy Preview https://deploy-preview-224--aria-practices.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@howard-e howard-e requested a review from alflennik May 31, 2023 14:41
@howard-e howard-e marked this pull request as ready for review May 31, 2023 14:41
@howard-e
Copy link
Contributor Author

howard-e commented May 31, 2023

I've verified this should render the images as expected, when deployed, with a local build of w3c/wai-website.

Copy link
Contributor

@alflennik alflennik left a comment

Choose a reason for hiding this comment

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

I actually like the approach of making a special case for this - after all, it's the only example of loading an image in this way. It's great to see this example working again.

Copy link
Contributor

@mcking65 mcking65 left a comment

Choose a reason for hiding this comment

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

This seems like a reasonable stopgap, but I'd like to include tracking of an issue to remove it. I think we will remove it when we have a fix on the content side. Not sure that is worthwhile to fix it on the build side ... supporting this capability is unnecessary complexity.

@mcking65 mcking65 changed the title Fix for broken feed display images Stopgap fix for broken feed display images Jun 5, 2023
@howard-e howard-e requested a review from mcking65 June 7, 2023 15:48
@mcking65
Copy link
Contributor

mcking65 commented Jun 8, 2023

Thank you @howard-e!

@ccanash ccanash merged commit b517f35 into main Jun 8, 2023
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.

4 participants