Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social media dashboard with theme switcher

Martaβ€’ 630

@martam90

Desktop design screenshot for the Social media dashboard with theme switcher coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi, Could you please have a look on BEM I used in the challenge? Does the name of the classes make sense? This is my 3rd project with BEM... Is there any other method how to code a rounded item inside the toggle button? I create <div> inside <button> How can I move the content to the middle of the page without cutting the top grey background on right and left side?

I would appreciate any feedback from you.

Thank you and Happy Christmas for all of you :)

Community feedback

P
ApplePieGiraffeβ€’ 30,565

@ApplePieGiraffe

Posted

Hey, Marta! πŸ‘‹

Merry Christmas! πŸŽ„ And nice job on this challenge! πŸ‘

I think your BEM looks fine! Perhaps box__small could be box--small, thoughβ€”since "small" seems more like an adjective (and therefore, a modifier rather than an element of box)β€”but that's a small thing! πŸ˜‰

Simply follow the helpful suggestions given by the others and I think you'll be good to go! πŸ‘

Keep coding (and happy coding, too)! 😁

1
Abhikβ€’ 4,840

@abhik-b

Posted

Hi Marta, Good job on this challenge ! I really liked it πŸ’―

Just some opinions ~

  • giving width to a element and then giving margin: 0 auto can easily center that element on X-Axis with respect to parent element
  • you can use a pseudo-element for white circle, make details__btn have position:relative , then give that pseudo element a position absolute ; top:50%; transform:translateY(-50%) to center it on y axis
  • then you can transform:translateX(some-value) to move the toggle button

Happy Christmas πŸ₯³

1
iMervinCβ€’ 215

@iMervinC

Posted

Great job! Looks great!

To center your content just add {margin: 0 auto} to your wrapper and apply the background color to the body.

Add check this out too. Might help with your toggle. (https://codepen.io/designcourse/pen/OGVZjr)

1

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord