ROCKY BARUA• 1,090
@Drougnov
Posted
@dombrga yes sir, it is close enough. You did a marvelous job.
Everything is perfect except the background pattern. It's not flexible enough for bigger screen.
You can fix it by adding background-size: contain.
Change the h2 to h1 to make your code accessible.
Plus you need to add a different pattern for mobile devices. But the design is pretty good. Great job. Happy coding :)
0
Dom B• 190
@dombrga
Posted
@Drougnov thanks for the suggestion! Just a question, how is h1 accessible and not h2 what is accessibility here? Thank you!
0
ROCKY BARUA• 1,090
@Drougnov
Posted
@dombrga Good question. For accessibility, we need to use heading in a sequence. Like this: h1>h2>h3 etc. So we can't start without a first-level heading.
0
