Skip to content
OverviewSolutions
2
Comments
3
Frank Lopes
@franklpjr

All comments

  • Anderson Silva•30
    @devspidersilvaSubmitted almost 2 years ago

    Sou novo como desenvolvedor front end. Usei para fazer esse desafio apenas, html e css. Aceito qualquer crítica construtiva.

    CSS GRID e FLEX BOX,

    2
    Frank Lopes•80
    @franklpjrPosted almost 2 years ago

    Hi Anderson! Your solution is very well. You can improve just the hover effects of navbar items and the three tittles down. You can also follow the style guide to select the text colors. Besides that, the site works perfectly in the mobile format. Another tip: try to use this space to practice your english like the most part of users. I'm Brazilian, but i try to communicate myself just in english to facilitate new opportunities.

    Hope you find this helpful ;)

    Marked as helpful
    0
  • Prince Adimado•120
    @primadoSubmitted almost 2 years ago

    Hello there, through this project I have been able to learn how to make a navigation menu with React. I made the website responsive as possible.

    But what I couldn't do was make the dropdown in the sidebar a dropdown as shown in the expanded mobile design. Kindly comment with your review of my work and possible solutions. I really want to learn, build and grow. Thank you.

    Intro section with Dropdown Navigation

    1
    Frank Lopes•80
    @franklpjrPosted almost 2 years ago

    Hi! You've done a very nice job! You can improve the text-align of the dropdown-menu to left and correct the features' icons. You also should attempt to the background-color of body and image and fonts' width. These details about colors and font-size are described on the style guide.

    I hope you find this helpful =)

    0
  • Frank Lopes•80
    @franklpjrPosted almost 2 years ago

    Hello! Congratulations, your work is great! You can improve the mouse hover effect in the three down tittle ("Reviving Retro PCs, Top 10 laptops 2022 and The Growth of Gaming..."). You used it on the bottom "read more" and on the navbar. I think you just forgot.

    cursor: pointer

    I hope you find this helpful.

    Marked as helpful
    1