Design comparison
SolutionDesign
Community feedback
- @ClaudiaRamirezDPosted 2 months ago
Good job! Remember that small details matter, such as capital letters and spacing.
0 - @jen067Posted 2 months ago
-
Did not write in Markdown. Writing in Markdown would help clearly identify any issues and the process of styling.
-
HTML
- Add Classes to Tags to Differentiate Elements
- Since the styles of p tags differ, adding a class can effectively apply paragraph styles to individual sections.
- Avoid Overusing ID: When there are many image elements, using id="image" is not a good choice. Switching to classes would allow for consistent styling across similar tags.
- Footer Setup: In my opinion, there are too many wrapping layers. Instead of using <div id="avname"><h4>greg hooper</h4></div>, it could be simplified to <h4>greg hooper</h4> with a class added to the h4.
- CSS
- font-family: 'figtree'; should be changed to 'Figtree', sans-serif;.
- Use rem instead of width, padding, and margin sizes to adjust according to the font size when the screen scales.
- For the footer, justify-content: flex-start; is the default style and does not need to be set additionally.
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