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

fix(gatsby-plugin-image): Pass sizes for source #29192

Merged
merged 5 commits into from
Jan 26, 2021
Merged

Conversation

laurieontech
Copy link
Contributor

This results in sizes on the source element. However, we're still testing to make sure it's appropriately picked up by the browser.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Jan 25, 2021
@laurieontech laurieontech added topic: media Related to gatsby-plugin-image, or general image/media processing topics and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Jan 25, 2021
@laurieontech laurieontech requested a review from ascorbic January 25, 2021 18:40
@ascorbic ascorbic merged commit 0ddbbfe into master Jan 26, 2021
@ascorbic ascorbic deleted the fix/source-sizes branch January 26, 2021 09:57
vladar pushed a commit that referenced this pull request Jan 27, 2021
* this should fix sizes for source

* update yarn lock

* let users pass via props

* optional chaining for the win

* fix tests

(cherry picked from commit 0ddbbfe)
@vladar
Copy link
Contributor

vladar commented Jan 28, 2021

Published in [email protected]

pragmaticpat pushed a commit to pragmaticpat/gatsby that referenced this pull request Apr 28, 2022
* this should fix sizes for source

* update yarn lock

* let users pass via props

* optional chaining for the win

* fix tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: media Related to gatsby-plugin-image, or general image/media processing topics
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants