Rajat Sardesai• 120
@rajatsardesai
Submitted
Feel free to give me any feedback for improvements. Ill be happy to hear that.
Old
@Old1337
All comments
- Old• 6,240
@Old1337
Posted
Hi there,
i believe that you have submitted the wrong challenge. re-submit the challenge so that can community review it
0 - Nisha• 180
@Nisha2018
Submitted
Please provide your valuable feedback :)
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of div to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
1 -
- Akira• 110
@dekitsuneco
Submitted
I'm a new to React so I would appreciate any criticism to improve my future works.
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of span to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
1 -
- Andriano• 110
@Katarsis-601
Submitted
Give me some criticism especially in jsx and scss code
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of h3 to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
1 -
- sanket kumar• 300
@sanketcharanpahadi
Submitted
Any suggestions or improvements are welcomed.
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
- put the quote inside a <blockquote> tag instead of h1 to be more semantic
i hope this is helpful and goodluck!
Marked as helpful
1 - Justin Gorelik• 100
@jgorelik23
Submitted
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of div to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
1 -
- Gerardo Saz• 140
@pgap22
Submitted
Any Feedback is appreciated
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
0 -
- Samuel Seve• 460
@nihilboy1
Submitted
Tentei fazer com next, ai deu problema e eu fiz com react normal mesmo. haha
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of h1 to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
0 -
- Macaulay Eden• 130
@EdenExperiments
Submitted
First attempt at using fetch within a project, have included commented code of several methods to ensure practice of them all.
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
-
add cursor pointer to button so that the user knows its clickable
i hope this is helpful and goodluck!
Marked as helpful
1 -
- clive• 140
@clivepato93
Submitted
always grateful for feedback
lets connect on twitter
@ciccio_ct93
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
0 -
- javiergranadosr• 170
@javiergranadosr
Submitted
Excelente para practicar, consumo de APIS
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
0 -
- Bashar Khdr• 930
@BasharKhdr1992
Submitted
Responsive Random Advice Generator using ReactJS, and fetch API.
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
-
chnage the height of the main element to 100vh
i hope this is helpful and goodluck!
Marked as helpful
0 -
- Manon cabret• 180
@GirafOnTrip
Submitted
Feel free to comment ;)
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
- put the quote inside a <blockquote> tag instead of p to be more semantic
i hope this is helpful and goodluck!
Marked as helpful
0 - Abbas Saad• 260
@AbbasSaad27
Submitted
Hi! It's been a while since I completed a challenge. I came again to brush off my skills a bit :D Any feedback would be really appreciated! :D
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of h1 to be more semantic
-
check the HTML report to fix accessiblity issues
-
add cursor pointer to button so that the user knows its clickable
i hope this is helpful and goodluck!
Marked as helpful
1 -
- Shann Timothy• 160
@ShannTimothy22
Submitted
All feedback will be well received. Thanks!
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
hope this is helpful
Marked as helpful
1 -
- Wissem• 180
@Wissemh
Submitted
Hello everyone, This is the very time I work with APIs. So any feedback on how I can improve my code is more than appreciative. Also, can someone help me on fixing the pause icon? Thanks in advance
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
0 -
@Norton794
Submitted
What could be modified in the code, so it would fit the corporate standards?
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
0-
- Bulaien• 110
@Bulaien
Submitted
Its CSS only because i have not the knowledge in JS.
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of h2 to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
0 -
- Daniel Kwon• 30
@danielkwon89
Submitted
- first time using media queries
- still don't know if I'm using flexbox best practices
Old• 6,240@Old1337
Posted
- add cursor pointer to button so that the user knows its clickable
Marked as helpful
1 - Guillermo• 170
@Guille2000
Submitted
I had a rough time doing the responsive but i think it's ok hahaha any feedback would be aprecciated!
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of div to be more semantic
-
check the HTML report to fix accessiblity issues
-
always wrap interactive elements such as caja__logo div inside a button element
-
add cursor pointer to button so that the user knows its clickable
i hope this is helpful and goodluck!
0 -
- Suprefuner• 470
@Suprefuner
Submitted
Hi All,
After some good comments I did some amendment, please check and please let me know if there is anything else I should change!!! Thanks so much!
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
0 -
@Nicholasjr
Submitted
I enjoyed working on this. Learnt a lot about API's, I'd use a more recent one like fetch on my next project
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
-
add cursor pointer to button so that the user knows its clickable
i hope this is helpful and goodluck!
0-
- elasri21• 1,280
@elasri21
Submitted
If you want some advice all you need is to click the button and then you will receive as many advice as you need Enjoy getting advice for free And give me your feedback it helps to grow and see my mistakes and errors thank you all.
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of P to be more semantic
-
check the HTML report to fix accessiblity issues
i hope this is helpful and goodluck!
Marked as helpful
1 -
- Jacks113• 170
@Jacks113
Submitted
Behaviour on different devices, further interactivity, responsives. Constructive criticism
Old• 6,240@Old1337
Posted
Hello there! 👋
Congratulations on finishing your challenge! 🎉
I have some feedback on this solution:
-
put the quote inside a <blockquote> tag instead of p to be more semantic
-
check the HTML report to fix accessiblity issues
-
add cursor pointer to button so that the user knows its clickable
i hope this is helpful and goodluck!
Marked as helpful
1 -
