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

feat: add eleventy-plugin-sharp, process thumbnails for ideas (resolves #46) #90

Merged
merged 7 commits into from
Sep 17, 2020

Conversation

greatislander
Copy link
Member

  • This pull request has been tested by running npm run test without errors
  • This pull request has been built by running npm run build without errors
  • This isn't a duplicate of an existing pull request

Description

This PR integrates eleventy-plugin-sharp and implements thumbnail generation for the ideas page.

Steps to test

  1. Visit Ideas page.

Expected behavior: Ideas have properly scaled thumbnails.

Additional information

Known issues: the shortcodes registered by this plugin are incompatible with our current preview mechanism. They also don't seem to work within partials at present. More to investigate.

Future enhancements: resize social images to match Open Graph recommendations.

Related issues

Resolves #46.

@greatislander greatislander added the enhancement New feature or request label Sep 17, 2020
@greatislander greatislander added this to the 1.3.0 milestone Sep 17, 2020
@greatislander greatislander self-assigned this Sep 17, 2020
@greatislander greatislander changed the title feat: add eleventy-plugin-sharp, process thumbnails for ideas (resolves feat: add eleventy-plugin-sharp, process thumbnails for ideas (resolves #46) Sep 17, 2020
@netlify
Copy link

netlify bot commented Sep 17, 2020

Deploy preview for idrc ready!

Built with commit fa25016

https://deploy-preview-90--idrc.netlify.app

@github-actions
Copy link
Contributor

github-actions bot commented Sep 17, 2020

Visual Regression Test Results

❌ 5 scenarios failed, ✅ 13 scenarios passed

Scenario Details ❌ Home (phone) — Before / After (2.57% mismatch)
❌ Home (tablet) — Before / After (3.94% mismatch)
✅ Home (desktop) — Before / After
✅ News (phone) — Before / After
✅ News (tablet) — Before / After
✅ News (desktop) — Before / After
❌ Ideas (phone) — Before / After (12.25% mismatch)
❌ Ideas (tablet) — Before / After (9.16% mismatch)
❌ Ideas (desktop) — Before / After (9.30% mismatch)
✅ Projects and tools (phone) — Before / After
✅ Projects and tools (tablet) — Before / After
✅ Projects and tools (desktop) — Before / After
✅ IDRC Consulting (phone) — Before / After
✅ IDRC Consulting (tablet) — Before / After
✅ IDRC Consulting (desktop) — Before / After
✅ Vision Technology Service (phone) — Before / After
✅ Vision Technology Service (tablet) — Before / After
✅ Vision Technology Service (desktop) — Before / After

To review the full report with screenshots, visit the Github Actions page and download the backstop-report artifact.

If these changes were intended, run npm run backstop:test-docker && npm run backstop:approve locally and commit your changes.

@greatislander greatislander merged commit 7a501dd into dev Sep 17, 2020
@greatislander greatislander deleted the feat/sharp branch September 17, 2020 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add eleventy-plugin-sharp for image processing
1 participant