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

New schema pictures / support mobile view #25

Merged
merged 4 commits into from
Mar 20, 2024
Merged

Conversation

manuhabitela
Copy link
Collaborator

Purpose

Show communs schema pictures correctly on mobile and update all pictures to new ones from Johann

Copy link
Contributor

@lebaudantoine lebaudantoine left a comment

Choose a reason for hiding this comment

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

Nothing related to the PR, but the second slider assets feel (a bit) weird. I'll discuss it with Johann, not a top prio btw

the desktop image is split in half for mobiles and set up with slides
the desktop image is split in half for mobiles and set up with slides.
We also add some screen reader only text to describe the images in a
semantic way.
- use a paragraph instead of a div so that screen reader support is
better (keyboard shortcuts better support paragraphs)
- use a non-breaking space before the exclamation mark to prevent a
lonely "!" when space is laking
- add a screen-reader specific heading so that screen reader users dont
mistakenly think this button is tied to the h3 just above
@manuhabitela manuhabitela merged commit 795cb03 into main Mar 20, 2024
3 checks passed
@manuhabitela manuhabitela deleted the feat/schemas branch March 20, 2024 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants