your anchor closing tag is wrong. remember closing tag is like this </a>
but a self-closing element like input would be like this <input /> without a closing tag just one tag
actually his code is fine with this, it doesnt matter which attribute comes first inside the elements whether its id or class or type or href or target.
that’s just your preference how you wanna order them
he’s having a problem with the task after this, please read the forum carefully so you wont confuse him more.
and please don’t just type the answer, it’ll be better for them to find it on their own just explain what you see wrong and let them do it