@amrajat
Posted
Congratulations on completing the challenge.
I want to suggest the below points please consider if you feel helpful.
- Why did you wrap this component within
content-wrapper
I feel it does just unnecessary nesting. - This component is not centered horizontally and vertically if can fix this easily with easy hacks such as margins, using Flexbox or grids.
- You did not choose the right breakpoint for the media query. always choose the breakpoint where the design starts breaking.
- max-width: 50% on .content is not a good value here. try something more meaningful here you could have used rems for ease instead of screen-based units %s.
- the box shadow seems a little dark here which is not looking realistic please consider reduced $alpha
You can see my solution for the same challenge.
https://www.frontendmentor.io/solutions/results-summary-component-html-css-sass-obzWVuf4OG
Happy coding :)
@Eyepop01
Posted
@amrajat thank you so very much for taking your time to point point out this correction... I appreciate... I'll work on it...