fix(ideal-image): do not pass down img prop

[lex111]

Pre-flight checklist

• I have read the Contributing Guidelines on pull requests.
• If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
• If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Screenshot below speaks for itself:

Test Plan

Preview

Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/

Related issues/PRs

[Josh-Cena]

In this case we don't need to destructure className or alt since they aren't overridden

[netlify]

✅ [V2]

Built without sensitive environment variables

Name	Link
🔨 Latest commit	90acd46
🔍 Latest deploy log	https://app.netlify.com/sites/docusaurus-2/deploys/6359694d2399970008353bb1
😎 Deploy Preview	https://deploy-preview-8250--docusaurus-2.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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

[github-actions]

⚡️ Lighthouse report for the deploy preview of this PR

URL	Performance	Accessibility	Best Practices	SEO	PWA	Report
/	🟠 59	🟢 97	🟢 100	🟢 100	🟢 90	Report
/docs/installation	🟠 79	🟢 100	🟢 100	🟢 100	🟢 90	Report

[slorber]

LGTM

Couldn't reproduce the extra img attribute on prod, is this only happening in dev mode?

[Josh-Cena]

We only use a plain <img/> tag in dev. In prod the actual ideal image component takes care of the prop.

[lex111]

We only use a plain tag in dev.

This is also used in the production site see

docusaurus/website/src/pages/showcase/_components/ShowcaseCard/index.tsx

Line 64 in 97c9710

`https://slorber-api-screenshot.netlify.app/${encodeURIComponent(

[slorber]

ah I see, only the ones with a serverless image, can repro now thanks 👍