Siddarth
@siddarthm93All comments
- @qvazi@siddarthm93
Congrats for completing the task, One suggestion is give a proper naming for class it will be best practice and other will understand the code easily.
Marked as helpful - @ARPIT73881What are you most proud of, and what would you do differently next time?
okay
What challenges did you encounter, and how did you overcome them?okay
What specific areas of your project would you like help with?okay
@siddarthm93congrats for completing the task, you have done good design
- @thedanielkingWhat are you most proud of, and what would you do differently next time?
I am proud of completing this task and i wish to start differently next time
What challenges did you encounter, and how did you overcome them?I couldn't place the first and last child of the card in the center but i finally found a way out by using transform: translateY(50%)
What specific areas of your project would you like help with?Help in learning React.js better and talks on "em" and "rem"
@siddarthm93Congrats for completing the task,
You have to keep the h1 size more .
Marked as helpful - @PrestonLTaylorWhat specific areas of your project would you like help with?
I would like any feedback on how I implemented my CSS stylings
@siddarthm93Congrats for completing the task, It was good work
- @mbd89What are you most proud of, and what would you do differently next time?
proud it looks great:)
What challenges did you encounter, and how did you overcome them?none
What specific areas of your project would you like help with?all is good
@siddarthm93congrats for completing the task, you have done good work.
One feedback on the mobile screen is try to achieve same design as in the image.
Happy Coding.
- @maurihed@siddarthm93
congrats on completing your task.
You have done a good work.
- @ishankgupta95What are you most proud of, and what would you do differently next time?
Will try to improve CSS
What challenges did you encounter, and how did you overcome them?None
What specific areas of your project would you like help with?Responsiveness
@siddarthm93Hi Ishank Gupta,
congratulations for completing the task, you have done a good work. i liked the way u used the css but one feedback was you have used the landmarks more which is unnecessary try to avoid it, as it is a card try to use accordingly like "div class="card" ,card-header, card-body and card-footer".
Marked as helpful - @mahmudhmhWhat are you most proud of, and what would you do differently next time?
tried some new CSS styling using root colors
What challenges did you encounter, and how did you overcome them?being able to make the qr in the center of the card
What specific areas of your project would you like help with?i think after review and feedback i can change a bit in the qr component
@siddarthm93I think its good but you need to add font family and check the design to match the same
Marked as helpful