Skip to content
OverviewSolutions
3
Comments
1
Ivan
@shin-bot87

All comments

  • Ivan•50
    @shin-bot87Posted over 2 years ago

    Hello Wallace!

    • I have downloaded your code and in my computer i can see it well, however in the miniature it's see the content out of the page. I have tested it with differents resolutions (1440px, 768px and 320px) and it's works well.

    • You got a little typo in your code. At the 16 line you have a div container named "main_container" wich doesn't have closed tag. Also you have a next div named "main_content" so it doesn't make sense surround twice the same content with two divs.

    • The title (where P E R F U M E it is) you have separated with white space in the HTML document. A good practise would be use letter-spacing in your CSS styles.

    • You have abuse of the specifity in your css stylesheet. For examplo, if you hava a class called "title" you can just use it like this in your css instead of ".main_container .main_content .info_section .title"

    The final result it's quite good =D it's really likes the original one.

    Everything wrote above was with all my best intention to help others to improve and get good practise and i never pretended to offend.

    I hope to helped you!

    Regards.

    Marked as helpful
    1