bootstram and css flexbox

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted over 2 years ago

  Hey there,

  Your solution looks good at first glance! There's not much to comment on the CSS as it's Bootstrap but there's a few bits you could perhaps take a look at:

  1. You could increase the font-size of the header links a bit, they're really hard to read at the moment.

  2. "image" isn't a helpful alt text. If you're unsure how to write better ones, I suggest you read this article.

  3. I'd double-check the email validation. I could get away by trying to submit "something@stuff", which definitely isn't a valid email address >.<

  4. You put aria-hidden on all your social icons. First of all, you should wrap them in a tags, they're links to social media websites. You shouldn't hide them, instead you could give them aria-labels that describe where those links lead.

  5. You could remove the width: 60% from the big intro image so it doesn't stretch too much.

  0

  ---

  

  patil sahil•440

  @Patilsahil821Posted over 2 years ago

  @FluffyKas Thanks you very much for suggestion

  0