Social media dashboard with theme switcher

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 4 years ago

  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-bPosted almost 4 years ago

  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

  @iMervinCPosted almost 4 years ago

  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