Hi @girish.2500 and welcome to the community
Great work. Congratulations!
This may have been a bit too much, because we are only marking up a letter. But I admit that’s something like a special case.
Regarding these structuring tags I have only a small remark: You used <header>
to wrap the heading elements (<h1>
, <h2>
). This is the wrong usage for <header>
. It’s for a higher level of structuring, mostly to mark the top part of a whole website. For example, here on Discourse the <header>
is used for the top bar with the Mozilla logo and the various buttons for search, menu and account settings.
Looking at all the other HTML tags, you nearly perfectly used all of them. The only thing is that you could also wrap the quote’s source on the last line in a <cite>
element. Everything else is fine.
I hope that helps.
Have a nice day,
Michael
PS: Your email address is visible in your post. It ended up in the optional “name” field of your profile. I recommend removing it.