Design comparison
Solution retrospective
Hi guys, I hope you are doing great. I just have one question: Why did my border-radius on the main not work?
And if you come across any issues in my code please let me know.
Thanks a lot to everyone Konstantin
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi Konstantin Christ, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
If we see how the layout is behaving at higher resolutions, with the help of google dev tools, we will see that it is stretching a lot, to solve this we can use a max-width with the value we want the content to stop growing and to center use margin: 0 auto;
Instead of putting the css, in the body, prefer to put it in a separate file! in larger projects, the css ends up taking up a lot of space!
The rest is great!
I hope it helps... 👍
Marked as helpful0 - @MahdiAljazairiPosted about 2 years ago
I just realized something that confused me for a bit. When you submitted your solution, you put a link to a commit instead of the link of the repository itself.
Click the three dots to the right of the name of the solution and choose 'Edit solution'. Then change the Repository URL field to (https://github.com/Konstantinchrist/3-col-responsive-site), and click Update Solution.
It's not that big of a problem, but it better be avoided in the future.
0 - @MahdiAljazairiPosted about 2 years ago
The corners of the
div
s inside ofmain
are just flowing outside. The most straightforward solution is adding "overflow: hidden;
" tomain
inorder to hide anything that overflow out of it.Also, I would suggest putting the styles in an external file, rather than in a
style
tag. This is better both for readability and maintainability.EDIT: Each property that
transition
applies to should be specified separately in a comma-separated list:button { transition: color padding border 150ms; /* Wrong */ transition: color 150ms, padding 150ms, border 150ms; /* Correct */ }
0@KonstantinchristPosted about 2 years ago@MahdiAljazairi thanks I didn't realize that
0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord