kasey• 320
@kaseyvee
Posted
Based on your media query, I assume you took a mobile-first approach, which is great!
Something I wish someone told me from the start was to use padding instead of margins where possible as it gets the job done with less code.
Here, I would remove margin-top and margin-left from .summary and instead add just use padding: 20px;.
For a smoother transition between mobile to desktop, .container width can be 100%.
For .result and .summary to align nicely for desktop, you can remove the height from .result and add justify-content: space-between instead.
Overall awesome job!
Marked as helpful
0
