Vanilla HTML, CSS Grid and Flexbox

Community feedback

• 

  Ali•260

  @alihuseynzade23Posted almost 3 years ago

  Hi Ahmed! Well done.

  -My only recomendation will be to use use "a" tag, not the "button" tag for the buttons. Because if you will create real sites(I hope you will), these buttons will have to redirect to another page and the tag "a" is best for it.

  Marked as helpful

  3

  ---

  

  P

  Ahmed Abdelaal•220

  @shrki416Posted almost 3 years ago

  @AleksHNZ awesome, hadn't considered that. Will make that change :)

  0
• 

  P

  David Turner•4,150

  @brodiewebdtPosted almost 3 years ago

  Nice job. This is well done. @AleksHNZ is right about the a tag. I give them a class of .btn and other classes for styling multiple buttons (btn-primary, btn-secondary, btn-ghost, etc.)

  Marked as helpful

  1

  ---

  

  P

  Ahmed Abdelaal•220

  @shrki416Posted almost 3 years ago

  @brodiewebdt Thank you! This is a great idea. I'll make those changes.

  0
• 

  P

  David Turner•4,150

  @brodiewebdtPosted almost 3 years ago

  Your welcome.

  0