Skip to content
OverviewSolutions
22
Comments
6
Ahmed Eid
@ahmeDeid67

All comments

  • Ahmed Eid•380
    @ahmeDeid67Posted about 3 years ago

    Very very thank you, For all these tips. I will put it in my mind Again thank you.

    0
  • Bayoura•105
    @BayouraSubmitted about 3 years ago

    Hello!

    I'm happy about any feedback you're willing to give. :)
    Also, I have one question: My report states that there is one accessibility issue (my project doesn't include an h1 tag). I'm not sure which element in this project would really 'qualify' as a heading. Could someone please explain what should be done in such cases?

    Thank you!

    Mobile First, HTML, CSS and Flexbox

    1
    Ahmed Eid•380
    @ahmeDeid67Posted about 3 years ago

    Nice work, Vector Crest uses it as h1.

    Marked as helpful
    1
  • Ahmed Eid•380
    @ahmeDeid67Posted about 3 years ago

    nice work, but increase the padding a little.

    Marked as helpful
    0
  • Ahmed Eid•380
    @ahmeDeid67Posted about 3 years ago

    Nice work, but you need to remove text-align from the heading, paragraph, and stats.

    Marked as helpful
    1
  • Ahmed Eid•380
    @ahmeDeid67Posted about 3 years ago

    increase the size of the heading, increase the width and line-height of the paragraph, increase the margin-top for the button.

    Marked as helpful
    0
  • Ahmed Eid•380
    @ahmeDeid67Posted about 3 years ago

    What is this ??

    0