@dvsanta
Latest solutions
Latest comments
- @Abeeujah
Hello, you submitted a nice solution and Here's some helpful tips to help you improve it
HTML :
Use the <main> tag to wrap all the main content of the page instead of the <div> tag. With this semantic element you can improve the accessibility of your page.
Use the <footer> tag to wrap the footer of the page instead of the <div class="attribution">. The <footer> element contains information about the author of the page, the copyright, and other legal information.
- @NarakornB@Abeeujah
Hello, you submitted a nice solution and Here's some helpful tips to help you improve it
HTML :
Use the <main> tag to wrap all the main content of the page instead of the <div> tag. With this semantic element you can improve the accessibility of your page.
Use the <footer> tag to wrap the footer of the page instead of the <div class="attribution">. The <footer> element contains information about the author of the page, the copyright, and other legal information.
Use appropriate header tags to give semantic value and improve accesibility in your page starting from <h1> - <h6>
Marked as helpful - @Nsohan@Abeeujah
Hello, you submitted a nice solution and Here's some helpful tips to help you improve it
HTML :
Use the <main> tag to wrap all the main content of the page instead of the <div> tag. With this semantic element you can improve the accessibility of your page.
Use the <footer> tag to wrap the footer of the page instead of the <div class="attribution">. The <footer> element contains information about the author of the page, the copyright, and other legal information.
Marked as helpful - @maxkdavisWhat are you most proud of, and what would you do differently next time?
What I would improve for next attempt:
-
Using CSS variables
-
Better understanding of Flexbox
-
Add Responsivness
@AbeeujahHello there 👋. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
HTML 📄:
Use the <main> tag to wrap all the main content of the page instead of the <div> tag. With this semantic element you can improve the accessibility of your page.
Use the <footer> tag to wrap the footer of the page instead of the <div class="attribution">. The <footer> element contains information about the author of the page, the copyright, and other legal information.
Always avoid skipping heading levels; Always start from <h1>, followed by <h2>, and so on up to <h6> (<h1>,<h2>,...,<h6>).
Alt text 📷:
Images Should Contain Alt Texts as it Helps Assistive Technologies tell the Value of the Image.
The alt attribute should explain the purpose of the image. Upon scanning the QR code, the user will be redirected to the frontendmentor.io website, so a better alt attribute would be QR code to frontendmentor.io If you want to learn more about the alt attribute, you can read this Article
Marked as helpful -
- @adgel1@Abeeujah
Hello here are some suggestions to help you improve your nice solution you have up there
HTML Accesibility:
Use the <main> tag to wrap all the main content of the page instead of the <div> tag. With this semantic element you can improve the accessibility of your page.
Use the <footer> tag to wrap the footer of the page instead of the <div class="attribution">. The <footer> element contains information about the author of the page, the copyright, and other legal information.
- @Kure-ru@Abeeujah
Hello, Nice Solution you've got.. Here's some tips to improve it
HTML :
Use the <main> tag to wrap all the main content of the page instead of the <div> tag. With this semantic element you can improve the accessibility of your page.
Use the <footer> tag to wrap the footer of the page instead of the <div class="attribution">. The <footer> element contains information about the author of the page, the copyright, and other legal information.
CSS : You can use Intrinsic Sizing or an accurate Extrinsic Sizing to handle Overflow on your component
Marked as helpful
