Mareeesh• 50
@Mareeesh
Submitted
Dolapo
@nofeesahdee
All comments
- Dolapo• 820
@nofeesahdee
Posted
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 @SAAJEVES
Submitted
Please check for possible parts to make improvement. Do check for it responsiveness also to different screens. Thanks
Dolapo• 820@nofeesahdee
Posted
You have done well saajeves
0- Paul Andres• 660
@paulpdoa
Submitted
Feedbacks are much appreciated!
Dolapo• 820@nofeesahdee
Posted
Hi 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 textin your images.
Goodluck
Marked as helpful
0 - Use a semantic tag such as
- aphrysof• 100
@aphrysof
Submitted
Any feedback would be highly appreciated.
Only Mobile and desktop responsive # Time tracking dashboard
#react#react-router#react-testing-library#sass/scssDolapo• 820@nofeesahdee
Posted
Hi Aphrysof, great work
However, to improve your accessibility, always include an
alt textinto your image.<img src = "image.png" alt = "image">0 - Nathan Leão• 30
@sinapsideo
Submitted
Please, I would love to hear some advices about how can I improve my work.
Dolapo• 820@nofeesahdee
Posted
Hi 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 helpful
1 - Francisco!• 255
@Francisco-Git99
Submitted
Hi, this is my solution, comments and feedback welcomed.
Dolapo• 820@nofeesahdee
Posted
Hi 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 helpful
1 - Matías Fuentes• 50
@LeoCaprile
Submitted
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!
Dolapo• 820@nofeesahdee
Posted
Hi Matias, you’ve done a great job. However, endeavor to include alt text into your images to improve accessibility.
0 - MANISHA SAMANTA• 50
@manishasamanta
Submitted
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 😊.
Dolapo• 820@nofeesahdee
Posted
Hi 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 helpful
0 - Please check your image format
- Camila Melo• 120
@IllaMelo
Submitted
Hello everyone!!! This is my response to the challenge and I would like to hear the feedback because I'm still a noob.
Dolapo• 820@nofeesahdee
Posted
Great work camila, you seem not to have any issues whatsoever. However, you probably forgot to change the background color.
Keep it up
Marked as helpful
1 - Lucas TAMAYA• 80
@LucasTamaya
Submitted
Hello, I would like a global opinion on my realization. Thank you in advance for your feedback!
Dolapo• 820@nofeesahdee
Posted
Hi Lucas, great work. To tackle the accessibility issue, you should wrap the body in a
<main>tagMarked as helpful
0 - Olaf Ziorko• 390
@OlafZiorko
Submitted
Goodevening or goodmorning whetever you are. Starting jurney with frontend technologies.If you can leavea comment and add something usefull please do it. Thanks to all of you.
Dolapo• 820@nofeesahdee
Posted
Hi Olaf, great job. However, you should include alt text into your image to improve accessibility
0 @Vidottizzz
Submitted
I found this challenge a little bit hard, but i completed it, the code is kinda messed up, can somebody tell me how can i improve this code?
Dolapo• 820@nofeesahdee
Posted
Hi Vidottizz,
Great job, here are few changes you might want to make to improve your code
- Always add alt text to your images to improve accessibility
- Ensure to use html5 semantics such as <header><main> this will help put your codes into landmarks
0- Udochukwu Amaefule• 570
@UDsGitHub
Submitted
Any feedback would be appreciated. As of now, the only question I can think to ask is how could I have approached this better.
Dolapo• 820@nofeesahdee
Posted
Hi Udo, great job
You've done well, however, since you use the <main> tag, using a class of main isn't necessary
0 - Maria Emoruwa• 50
@DanglingGem
Submitted
I had issues putting the right social media icons, can anybody help me with a few better explanations? Thank you.
Dolapo• 820@nofeesahdee
Posted
You are welcome
0 - Maria Emoruwa• 50
@DanglingGem
Submitted
I had issues putting the right social media icons, can anybody help me with a few better explanations? Thank you.
Dolapo• 820@nofeesahdee
Posted
Hi Maria great job, Here are few observations I think you might find helpful
- Always include alt text in your image to improve accessibility
- Your input is overflowing its container, you might want to add some padding to that
- You can not use id in multiple places, use class instead
- Add circle to the fab fa-circle-facebook to include circle in your icon
0 - Hanif Adams• 150
@omeizahanif
Submitted
Feedbacks would be appreciated.
Dolapo• 820@nofeesahdee
Posted
Hi, I think you should try as much to center your container.
use: body{ display:flex; justify-content:center; align-items:center; min-height:100vh; }
Marked as helpful
0
