Improve general accessibility #2

merged
opened by vielle.dev targeting main

Listing changes from top of commit to bottom:

  • Prettier code formatting.
  • Remove unneeded !importants
  • Move around where margins are placed to be a little more sane
  • Add a max-width to <p> tags of 60ch. This is a comfortable reading width in general and is ~= to the manual reading width from before. Also remove the manual <br> and <pre> linebreaks as they make formatting a bit weird and inconsistent on mobile
  • Add a max-width of 40ch to hr instead of a width of 40%. It'll be full width for smaller screens (instead of small and piddly), and wont blow out to be massive on larger screens. 40ch looks pretty alright with the 60ch character width.
  • Move the <pre> to only cover the ascii art so that paragraphs are consistent.
  • Wrap the <pre> and intro paragraph in <header> bc ✨accessibility✨
  • Wrap the main content in <main> bc ✨accessibility✨
  • Wrap each part in <section>. Why? You guessed it: ✨accessibility✨
  • Make the user list an ordered list and add some Loading placeholder text. This is hidden by the
0
by vielle.dev 1 comment
expand 1 commit
edc3b33b
improve the accessibility of the site
pull request successfully merged
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 2
AT URI
at://did:plc:4zht3z4caxwrw3dlsybodywc/sh.tangled.repo.pull/3m4vrx5dbqp22