Skip to content
OverviewSolutions
2
Comments
2
Cody
@CodyyLee

All comments

  • Tuan Linh•180
    @TuanLinh02Submitted about 2 years ago

    Thank you for watching.I can't put the word "5 end-game strategies to increase your win rate" on 2 different lines like in design.Can you show me?Thank you

    Notification page

    1
    Cody•50
    @CodyyLeePosted about 2 years ago

    Hey Tuan!

    It looks like you have flex on the element itself instead of on its container. :)

    .user-content--above{
    display: flex;
    flex-wrap: wrap;
}

    Try applying these styles onto its parent container instead and see if that helps resolve the issue.

    Hope this helps! :)

    1
  • Cody•50
    @CodyyLeePosted about 2 years ago

    Hey Norbert!

    Looks like you're on the right track! The layout of the page is looking close, and that is awesome. Looking at your GitHub, it seems you didn't include the assets folder and so it is only showing the alt tags. I'd recommend pushing them to GitHub so everyone can see the page in all of its glory!

    I'd also adjust the colors that were used for the body background and the 'New notifications' background. They're a bit darker than the design's. Another thing to implement would be the font-family used in the design as well as some font-weights to get it as close to the design as possible.

    Finally, it doesn't look like you're able to toggle the notification state's. I'd create a .js file to include this functionality.

    Great work! :)

    0