Dolapo
@nofeesahdeeAll comments
- @MareeeshSubmitted 6 months ago@nofeesahdeePosted 6 months ago
Your solution looks good beside the provided design, but a tiny colour change can be made to the border. Accessibility is great as well. Well done
1 - @Kevin-KigothoSubmitted almost 3 years ago
Please give any feedback, I will highly appreciate it.
@nofeesahdeePosted almost 3 years agoGreat job Kevin, For the accessibility, instead of using div, try use semantic html5 such as main
Marked as helpful1 - @SAAJEVESSubmitted almost 3 years ago
Please check for possible parts to make improvement. Do check for it responsiveness also to different screens. Thanks
- @paulpdoaSubmitted almost 3 years ago
Feedbacks are much appreciated!
@nofeesahdeePosted almost 3 years agoHi paul, great job.
Here are adjustments you can make to improve accessibility;
- Use a semantic tag such as
<header> <main>
to put your contents in landmark - Include the contents of your web page aside the
<header> and <footer>
into the<main>
tag - Always use an
alt text
in your images.
Goodluck
Marked as helpful0 - Use a semantic tag such as
- @aphrysofSubmitted almost 3 years ago
Any feedback would be highly appreciated.
@nofeesahdeePosted almost 3 years agoHi Aphrysof, great work
However, to improve your accessibility, always include an
alt text
into your image.<img src = "image.png" alt = "image">
0 - @sinapsideoSubmitted almost 3 years ago
Please, I would love to hear some advices about how can I improve my work.
@nofeesahdeePosted almost 3 years agoHi Nathan, you’ve done a great job. However, you can improve on accessibility by:
Using <main> tags instead of <div> Format the <img> url properly
Marked as helpful1 - @Francisco-Git99Submitted almost 3 years ago
Hi, this is my solution, comments and feedback welcomed.
@nofeesahdeePosted almost 3 years agoHi Francisco, Great work 😊, here’s a little feedback to improve accessibility.
Use ‘<main>’ tag to include all the other tags instead of <div>, basically what I am saying is, try as much as you can to use html5 semantics where necessary
Marked as helpful1 - @LeoCaprileSubmitted almost 3 years ago
Hi Guys! I made this in like 8 hours, in my spare time. I been learning React for like one month, can you give me feed back about the folder arquitecture that I used. Or if i used hooks wrong etc. Thanks!
@nofeesahdeePosted almost 3 years agoHi Matias, you’ve done a great job. However, endeavor to include alt text into your images to improve accessibility.
0 - @MizAndhreSubmitted almost 3 years ago
JavaScript is kind of hard for me right now, but I'm not giving up. Slowly but surely I will conquer it!
- @manishasamantaSubmitted almost 3 years ago
I faced problems to place the elements properly. is my approach correct? or I should change the way for positioning ? And please let me know about my mistakes 🙏🙏🙏. Thanks 😊.
@nofeesahdeePosted almost 3 years agoHi Manisha, great work.
Here are a few suggestions to help improve your accessibility and HTML issues;
- Please check your image format
<img src = "./images/....>
not `<.\image...> - Ensure to wrap your body into a
<main>
tag. - Use semantic HTML tags as much as possible to replace divs
Keep it up, it gets better.
Marked as helpful0 - Please check your image format
- @IllaMeloSubmitted almost 3 years ago
Hello everyone!!! This is my response to the challenge and I would like to hear the feedback because I'm still a noob.
@nofeesahdeePosted almost 3 years agoGreat work camila, you seem not to have any issues whatsoever. However, you probably forgot to change the background color.
Keep it up
Marked as helpful1 - @LucasTamayaSubmitted almost 3 years ago
Hello, I would like a global opinion on my realization. Thank you in advance for your feedback!
@nofeesahdeePosted almost 3 years agoHi Lucas, great work. To tackle the accessibility issue, you should wrap the body in a
<main>
tagMarked as helpful0