FYLO dark theme layout

Community feedback

• 

  Aakash Verma•6,710

  @skyv26Posted almost 3 years ago

  Hi! Daniel,

  To solve accessibility issues:

  1. <a> should have an aria-label ... [Click here to read more here (https://dequeuniversity.com/rules/axe/3.5/link-name)

  2. Never left alt attribute empty in img tags Or add alt attribute in img tag, and add some meaning text in it, so that user able to understand.

  Happy New Year ;)

  Marked as helpful

  1
• 

  Sam Guliker•540

  @Sam-GulikerPosted almost 3 years ago

  Hi Daniel,

  I don't really have much to add to this comment ^^^^. But try to focus on spacing and paddings a bit more. It seems a bit off.

  Happy coding.

  1

  ---

  

  Daniel•120

  @dvenomb98Posted almost 3 years ago

  @Sam-Guliker yea i think its good idea to define padding and margin as variable :D thanks for reply

  1