Skip to content
OverviewSolutions
6
Comments
4
P
justine1607
@justine1607

All comments

  • justine1607•110
    @justine1607Posted about 1 month ago
    1. for me i recommend not to use height on the text for dynamic, because when the content changes it's gonna be mess because for example the contend has exceeded from the height you add it will be not good.
    2. don't set px instead use relative units like em, rem.
    3. your margin-bottom to the hero-title could be remove and you just add gap to the hero-container so the hero-title and the hero text will have consistent gap.
    0
  • justine1607•110
    @justine1607Posted about 1 month ago

    nice its responsive.just wanna leave a comment your all div and other element should be inside the main like body-main-your div's.

    Marked as helpful
    0
  • justine1607•110
    @justine1607Posted about 1 month ago
    1. border-top-style: outset; / / border-top-color: var(--cyan); / / border-width: 0.25rem; --this can be achieved by just one line of code try setting it to border-top: 5px solid var(--cyan);
    2. Use margin-left: auto instead. This will dynamically push the image to the right and is more flexible. Avoid using fixed values for margins unless absolutely necessary.
    0
  • justine1607•110
    @justine1607Posted about 1 month ago

    i like how you manage to use different images in different screen sizes.

    0