@sonickonic
Posted
Congrats on submitting your first solution! it looks amazing!
You've currently got two h1 elements, it will be better to use it once as it can cause accessibility issues. Two ways to solve it:
- Wrap all of that text as a single h1 and use a span element inside it to separate the text.
- Use h1 for the main heading and h2 for the sub-title.
Although it is a pixel perfect solution for the wide-screen, the mobile version is slightly off. Try next time to use min-width media queries instead of max-width. Starting from the mobile-first can lead to more simple solution)
Good job!
Hi, @sonickonic. Thanks for your kind suggestion.
Since I'm new to programming, I'm trying to keep practising, gaining knowledge and trying to explore more new kinds of stuff.
@sonickonic
Posted
@aayush182 for someone new to programming, You nailed it! Did you choose already your next challenge?
@sonickonic thank u again, and so far I haven't chosen anything.