Hand in Portfolio Technigo #42
Conversation
…nent responsiveness
src/sections/Skills.jsx
Outdated
| text-align: center; | ||
| width: 100%; /* Mobil (standard): Staplad layout */ | ||
|
|
||
| @media (min-width: 900px) { |
There was a problem hiding this comment.
You have the same media query 3 times in this styled.div. Try to place the media query at the bottom of the styling section and have only one :)
There was a problem hiding this comment.
Woops! :D I put together two of them but I have to keep the ul::after media query since its inside the ul :)
There was a problem hiding this comment.
No worries!
You can put the media query around the ul as well. Just write it again :)
ul::after {
...
}
@media (min-width: 900px) {
...
ul::after {
...
}
}There was a problem hiding this comment.
Great point :) I think I fixed it now :)
Npahlfer
left a comment
Npahlfer
left a comment
There was a problem hiding this comment.
Would have been nice to see the use of a theme file for colors, media queries and such. The padding in mobile is a bit on the large side, but overall good job! :)
|
Great idea! :) |
2 participants
Here is my Portfolio assigmnet!
https://portfolio-technigo.netlify.app/