Skip to content
OverviewSolutions
6
Comments
5
VADER900000
@VADER900000

All comments

  • VADER900000•60
    @VADER900000Posted about 9 hours ago

    use more semantic html pls looks accessible on mobile and pc layout looks good all good except u need more semantic html use

    0
  • VADER900000•60
    @VADER900000Posted 1 day ago

    code is well structered its accessible layout is good but the solution was a bit shorter and used a different font

    1
  • shaikh zayed•60
    @undersigned-devSubmitted 1 day ago
    What specific areas of your project would you like help with?

    Any review would awsm☺️

    flexbox

    1
    VADER900000•60
    @VADER900000Posted 1 day ago

    it looks accessible on pc the layout is good code well structured but it really differs from solution u shouldnt apply to urself but to the solution

    0
  • VADER900000•60
    @VADER900000Posted 1 day ago

    please fix the image improve or remove the footer it should be a little bigger like it is on the design

    0
  • Stefan Zorbas•50
    @stezorSubmitted 5 days ago
    What are you most proud of, and what would you do differently next time?

    I'd probably use different semantic element for text, specifically first "heading" element.

    What challenges did you encounter, and how did you overcome them?

    Most difficult part were git and github because I have scarcely any experience with either.

    What specific areas of your project would you like help with?

    I think I did well enough although all suggestions are welcome.

    Basic HTML & CSS

    1
    VADER900000•60
    @VADER900000Posted 1 day ago

    id make it a bit bigger

    0