@mattstuddert
Posted
Great work on this challenge, Gediminas! You've done a really good job. How did you find it working on a non-newbie project? Did you learn anything new while completing it?
Here are a couple of small pointers on adjustments I'd recommend for the HTML:
- You're writing uppercase "FOLLOWERS", etc in your HTML. I'd recommend not doing this, as some screen reader software will read this out letter-by-letter, therefore, making the content inaccessible to screen reader users. Instead, I'd write it normally in the HTML and then use
text-transform: uppercase;
in your CSS to visually alter it. - You're using a paragraph tags for your
. social__title
and.subtitle-text
elements, whereas I'd say these are headings. Personally I'd use ah2
for the. social__title
andh3
for the.subtitle-text
to provide a proper content hierarchy. This is another crucial element in creating accessible web pages.
I hope this helps. Keep up the great work! 👍
@Senatrius
Posted
@mattstuddert It definitely felt more challenging. Things like data-* attributes haven't even crossed my mind before. Same with the toggle using CSS, things like using a hidden checkbox to create visual effects was one of those things I wasn't fully aware of. While in this case it was just a toggle, I'm sure I'll remember some of these techniques in other situations as well. So I'd say I learned quite a lot just from this one challenge :)
As for the uppercase HTML, I wasn't aware that capital text would mess with the screen readers, I'll be sure to keep that in mind.
Thanks for the advice!
@mattstuddert
Posted
@Senatrius you're welcome! Sorry for my delayed response to this. That's awesome to hear you learned a lot while going through this challenge!