Responsive design

Community feedback

• 

  Doston Nabotov•950

  @dostonnabotovPosted over 1 year ago

  Hi, there! 👋

  Congrats on completing the challenge. I inspected your code a little bit and here's solution to your problem with the "connection" along with some feedback:

  1. Wrap the <section> elements in another <div> element. Give it a class name, like "result-summary"
  2. Move your <main> CSS styles to that <div class="result-summary">
  3. Done!

  Also, it's highly recommended not to use id as a selector in your CSS. Stick with class names.

  Last, but not least, I recommend you check out the way Kevin Powell did this challenge. I bet you'll learn a lot of cool stuff from him.

  Good luck on your coding journey!

  Marked as helpful

  0
• 

  Pavel B.•270

  @JagholinPosted over 1 year ago

  Good job! I only have 1 suggestion:

  • people usually use class selectors, not id selectors in CSS. Here it doesn't really matter, but having class selectors allows for easier reuse of styles(ids have to be unique, classes do not).

  To fix your boundary between the 2 sections all you have to do is move your border-radius and box-shadow properties from #score to main

  Marked as helpful

  0