@attia-mahmoud
Posted
Hey Goorezy! I really like the background images and the theme-switch additions, it really gives life to the project! And I see no problems with the responsive design. Overall, you did a great job!
Regarding your problem, a really simple solution would be to add overflow: hidden
to the body element. This would prevent triggering the scrollbar due to any clipping outside the main viewport.
A tip for best practices would be to avoid using class selectors for elements that appear only once on the page, such as 'body', 'opbg', and 'card', but instead set them as ids, as classes are meant for elements that appear more than once.
Also, you can use the footer
element directly instead of creating a div with the class 'footer'.
Hope this helps!
Marked as helpful