@DammyD
Latest comments
- @suhaib52
You did it in a very good manner. Your code is well structured and readable.
It would be nice if you start using semantic html tags, these tags would reduce the accessibility issues.
You can also use :root to define the colors and then call them by using var() in your stylesheet. When you use the colors as a variable than your css code will look more precise.
Marked as helpful - @almeida883@suhaib52
You did it in a very good manner. Your code is well structured and readable.
It would be nice if you start using semantic html tags, these tags would reduce the accessibility issues.
You can also use :root to define the colors and then call them by using var() in your stylesheet.
I do not understand why you create separate style sheet for mobile and desktop.
Marked as helpful - @sawsan-code@suhaib52
Good job! Your code is well structured and readable. For html, try to use semantic html tags. Semantic html tag would help to improve the accessiblity.
Marked as helpful - @stefan123283@suhaib52
It would be better if you use the semantic html tags because these tags would reduce the accessibility issues.
You can also define the body in more details to make it more better such as; body { width: 100%; height: 100vh; display: flex; align-items: center; justify-content: center; font-family: 'Karla', sans-serif; background-color: var(--grayishBlue); font-size: 16px; }
Also use the margin 0 auto on the card id so it should be in the center of the page.
It would be nice if you use classes more as compared to the ids.
Marked as helpful - @kathrryn1@suhaib52
Your code is well structured and easily readable. Good job! You can define the colors by using :root at the start of the style sheet and then call them by using var().
Marked as helpful - @myrojoylee@suhaib52
I think you need to use the position relative and then use bottom property to move the profile picture.
