Skip to content

Tilde update structure css #30

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

Open
wants to merge 104 commits into
base: main
Choose a base branch
from

Conversation

tildetilde
Copy link

No description provided.

oskarnordin and others added 30 commits March 26, 2025 16:41
Added startQuiz button on top of the banner img.
tildetilde and others added 29 commits April 1, 2025 18:26
Added logo and changed padding to adjust height
- Optimized images.
- Made 2 + 2 cards.
…eader when the user wants to get back to main
added scrollIntoView on backtomain-button, now it scrolls up to the h…
This reverts commit ecfe5de, reversing
changes made to cd97847.
changed size on h2 in result page
…ts-html

updated about semantics and styling
Copy link
Collaborator

@NiklasAbrahamsson NiklasAbrahamsson left a comment

Choose a reason for hiding this comment

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

  • Nice addition of accessibility-related facts — educational content like this helps raise awareness and build user understanding.
  • Well done on the skip button and a clear tab order, both of which enhance keyboard navigation and screen reader usability.
  • Focus trapping works as intended, and keyboard navigation is seamless across components.
  • The feedback system — using both color and text — is a thoughtful and inclusive touch, ensuring all users receive the information they need.
  • The color palette in dark mode is well chosen and visually pleasant.
  • Overall structure and layout of elements are clear and well-organized.
  • Color contrast ratios meet accessibility standards.
  • Great use of both standard labels and ARIA labels, improving screen reader support.
  • Inclusion of a reduced motion option is excellent — it supports users with motion sensitivity and respects system preferences.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants