Frontend Mentor | Cody's profile

All comments

@TuanLinh02

Submitted almost 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

Cody• 50

@CodyyLee

Posted almost 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! :)

Norbert Dumashie• 30

@Tsidi

Submitted almost 2 years ago

All feedback is welcome

notification-submit-solution

Cody• 50

@CodyyLee

Posted almost 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! :)