Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

#2287: Fix for rounding error on src set #2300

Merged
merged 3 commits into from
Apr 6, 2020
Merged

#2287: Fix for rounding error on src set #2300

merged 3 commits into from
Apr 6, 2020

Conversation

larsroettig
Copy link
Member

@larsroettig larsroettig commented Mar 31, 2020

Description

Fix for rounding issue #2287

Related Issue

Closes #2287

Acceptance

Verification Stakeholders

Specification

Verification Steps

  1. Go to the FOO page.
  2. Verify the BAR shows up.
  3. Make sure BAZ does a thing.

Screenshots / Screen Captures (if appropriate)

Checklist

  • I have updated the documentation accordingly, if necessary.
  • I have added tests to cover my changes, if necessary.

@larsroettig larsroettig requested a review from jimbo April 1, 2020 06:44
@dpatil-magento dpatil-magento added the CI_APPROVED This PR should be included in the CI process. label Apr 2, 2020
@PWAStudioBot
Copy link
Contributor

PWAStudioBot commented Apr 2, 2020

Fails
🚫 Missing information in PR. Please fill out the "Verification Steps" section.
Messages
📖

Access a deployed version of this PR here. Make sure to wait for the "pwa-pull-request-deploy" job to complete.

📖 DangerCI Failures related to missing labels/description/linked issues/etc will persist until the next push or next nightly build run (assuming they are fixed).

If your PR is missing information, check against the original template here. At a minimum you must have the section headers from the template and provide some information in each section.

Generated by 🚫 dangerJS against 0055cab

@tjwiebell tjwiebell added the version: Patch This changeset includes backwards compatible bug fixes. label Apr 3, 2020
Copy link
Contributor

@jimbo jimbo left a comment

Choose a reason for hiding this comment

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

@larsroettig Thanks for the contribution. Good catch here, and the fix looks straightforward.

@dpatil-magento dpatil-magento merged commit 1efd508 into magento:develop Apr 6, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CI_APPROVED This PR should be included in the CI process. Partner: TechDivision partners-contribution pkg:venia-ui version: Patch This changeset includes backwards compatible bug fixes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bug]: The generateSrcset can generate broken images
6 participants