sudhan333• 10
@sudhan333
Submitted
md_mark
@mohdahsanrazakhan
All comments
- md_mark• 180
@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code.
You should add
lang="en"attribute inside yourhtmltag to improve accessibility and organization of your page.I hope you find this helpful.
Happy coding!
0 - Eivan• 50
@eivanliongson
Submitted
My solution to the NFT Component Preview
Did it similar to the design. Bummed because I can't figure out how to do the hover effect. Maybe someone could help me figure it out.
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code.
You should write
_blankinstead of just_or remove yourtargetattribute. But the best practice is to usetarget="_blank"inside youratag.I hope you find this helpful.
Happy coding!
1 - Hadi Farbakhsh• 40
@HadiFarbakhsh
Submitted
Hi, I'd be happy to have your comments on how to change the cursor type to be exactly like the challenge. Thankful
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code that I believe it will great interest to you.
HTML: This solution generates accessibility error reports. So fix it by one
h1tag inside your<article class="name">tag and inside<section class="description">bound the content insideh2-h6tag orptag in your index.html file to improve accessibility and organization of your page.I hope you find this helpful.
Happy coding!
Marked as helpful
0 - vanamos12• 10
@vanamos12
Submitted
How to center a box vertically?
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code that I believe it will great interest to you.
HTML: This solution generates accessibility error reports. So fix it by replacing the
<div class="main">element with the semantic element<main>in your index.html file to improve accessibility and organization of your page.You should add one
h1tag inside yourmaintag and addalt="QR code image"attribute inside<img src="images/image-qr-code.png">In place of
<div class="attribution">you should usefootertag.I hope you find this helpful.
Happy coding!
Marked as helpful
1 - ichimon-nashi• 120
@ichimon-nashi
Submitted
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code that I believe it will great interest to you.
HTML: This solution generates accessibility error reports. So fix it by replacing the
<div class="box-outer">element with the semantic element<main>in your index.html file to improve accessibility and organization of your page.You should add one
h1tag inside yourmaintag and addalt="QR code image"attribute inside<img src="images/image-qr-code.png">In place of
<div class="attribution">you should usefootertag.I hope you find this helpful.
Happy coding!
Marked as helpful
0 - BK Madsen• 60
@MadsenBK
Submitted
I dont have any specific questions for this challanege however any general feedback or errors Ive missed would be appreciated. Thank you!
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code that I believe it will great interest to you.
HTML: This solution generates accessibility error reports. So fix it by replacing the
<div id="mainContainer">element with the semantic element<main>in your index.html file to improve accessibility and organization of your page.Replace
h2tag toh1tag inside yourmaintag and addalt="QR code image"attribute inside<img src="images/image-qr-code.png">In place of
<div class="attribution">you should usefootertag.I hope you find this helpful.
Happy coding!
Marked as helpful
1 - SorinSr4• 60
@SorinSr4
Submitted
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code that I believe it will great interest to you.
HTML: This solution generates accessibility error reports. So fix it by replacing the
<div class="container">element with the semantic element<main>in your index.html file to improve accessibility and organization of your page. You should replaceh4tag toh1tag inside yourmaintag.I hope you find this helpful.
Happy coding!
Marked as helpful
0 - Umer Sohail• 80
@callmeUmer
Submitted
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code that I believe it will great interest to you.
HTML: This solution generates accessibility error reports. So fix it by replacing the
<div class="container">element with the semantic element<main>in your index.html file to improve accessibility and organization of your page. Addalt="QR code image"attribute inside<img src="images/image-qr-code.png" width="256">I hope you find this helpful.
Happy coding!
Marked as helpful
0 - Mahesh Paul• 30
@CityIsBetter
Submitted
md_mark• 180@mohdahsanrazakhan
Posted
Hello there 👋. Congratulations on successfully completing the challenge!
I have recommendations regarding your code that I believe it will great interest to you.
HTML: This solution generates accessibility error reports. So fix it by replacing the
<div class="container">element with the semantic element<main>in your index.html file to improve accessibility and organization of your page and replace the<div class="attribution">element with the semantic element<footer>in your index.html file. Addalt="QR code image"attribute inside<img src="./images/image-qr-code.png">I hope you find this helpful.
Happy coding!
1
