Skip to content
OverviewSolutions
15
Comments
2
Troneci Andrei
@TroneciAndrei

All comments

  • Troneci Andrei•260
    @TroneciAndreiPosted over 1 year ago

    Nice work, I like that you did

    Next time when you have to center a component try to put the display flex directly on the body if you do that you have more chances not to put multiple tags inside each other

    Here if you put flex on the body you can remove the main and use only the section tag for just this component is fine but you have more boxes you will end up with many tags inside each other and the html become hard to read

    Keep up with the learning and enjoy the process!

    Marked as helpful
    0
  • Troneci Andrei•260
    @TroneciAndreiPosted over 1 year ago

    Excellent work 👍 keep up and enjoy what you have learned!

    The layout seems to look good on all the screen size and I like how you structured the code and the fact that you use vite and not create-react-app that is a good thing

    The others already told you what you change I don't see anything else!

    1