JimTK16• 370
@JimTK16
Posted
Hi, some of my suggestions for improvement are:
- Avoid setting fixed height. An example is that for .container, you can remove the setting height: 60vh.
- Don't set fixed width for element when unnecessary. You set width: 500px for .summary, I think this prevent the element to be responsive.
- Set media query at your container width (48vw) instead of 375px.
Link to my github code for this project: https://github.com/JimTK16/results-summary-component
Marked as helpful
1
