Skip to content
This repository has been archived by the owner on Jul 26, 2023. It is now read-only.

fix reference to Material UI #1

Closed
wants to merge 1 commit into from

Conversation

oliviertassinari
Copy link

@oliviertassinari oliviertassinari commented Jul 7, 2023

<span class="text-center">React</span>
</li>
<li class="flex flex-col items-center justify-end">
<img src="./material-ui.png" alt="MUI" />
<span class="text-center">MUI</span>
<img src="./material-ui.png" alt="" />
Copy link
Author

Choose a reason for hiding this comment

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

Fix an a11y issue, this is a decorative image, it must have an empty alt, https://www.w3.org/WAI/tutorials/images/decorative/

<img src="./typescript.png" alt="Typescript" />
<span class="text-center">Typescript</span>
<img src="./typescript.png" alt="" />
<span class="text-center">TypeScript</span>
Copy link
Author

Choose a reason for hiding this comment

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

Wrong casing.

@@ -278,7 +278,7 @@
};</code
></pre>
<img
src="./Screen_ee.png"
src="./screen_ee.png"
Copy link
Author

Choose a reason for hiding this comment

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

The upper case is inconsistent with the other image. Also since only Linux is case-sensitive, this creates problems on macOS and Windows.

@slax57
Copy link

slax57 commented Jul 11, 2023

Thanks for the PR 👍

However this repo actually contains an old version of the landing pages (it was used as a draft).

The current version is here: https://github.com/marmelab/react-admin/tree/gh-pages/landing-page

Could you change the PR to target this repo/branch/folder instead?

Thank you so much

@oliviertassinari
Copy link
Author

oliviertassinari commented Jul 12, 2023

ok, I'm moving this marmelab/react-admin#9096

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants