namborghinii
@namborghiniiAll comments
- @omerturkerwebSubmitted over 1 year ago@namborghiniiPosted over 1 year ago
Good job on completing the challenge! :)
A reminder for next time: Group the number 1-5 by giving them the same name. By that the radio button will only be selected once.
Marked as helpful1 - @Tamana123-2Submitted over 1 year ago
In this project, I had problem with the image's border radius although I made it the same as expected, but the border-bottom-left and border-bottom-right were getting extra range. Any recommendation regarding to the project would be appreciated, plus I have started Java script but I do not know how to go on except from (w3schools) can you recommend me any other fun and useful source? Thanks!
@namborghiniiPosted over 1 year agoGood job on completing the challenge!
Instead of
border-radius: 1.5rem 1.5rem 1.5rem 1.5rem;
you can just writeborder-radrius: 1.5rem;
and it will be valid for all corner.If you want to practice your JS Skills (or other languages) you should definetely check out codewars.com :))
I hope I could help you. Happy Coding!
0 - @Samuel-MakindeSubmitted over 1 year ago
feedback welcome
@namborghiniiPosted over 1 year agoGood job on completing the challenge!
Please make sure to include font-family in your CSS as well as using the right margin for all elements. A little reminder for the next time: Make a responsive Design with media queries or responsive layouts. :))
HAPPY CODING!!
0 - @MichalKarczewiczSubmitted over 1 year ago
I just finished my second challenge. I want your suggestions what can be improved and in general what you like or you don't like. Thank you in advance for feedback :)
@namborghiniiPosted over 1 year agoGood job on completing the challenge! :)
Please remember to set font-family as well as font-weight for the different kind of sections.
Marked as helpful1 - @victordlh29Submitted over 1 year ago
What did you find difficult while building the project? I found the css difficult I defend myself but I think I have to improve much more
@namborghiniiPosted over 1 year agoJust take more time for the challenge next time and you'll got it! :)
Marked as helpful1 - @suyog-bhandariSubmitted over 1 year ago
Feel free to give a feedback...
@namborghiniiPosted over 1 year agoGood job on completing this challenge so far! :) Little reminder to center the bottom text and decrease the scale of the component in general so it fits on the screen without scrolling. Also add media queries for responsive Design and a box-shadow and you should be fine.
Marked as helpful2 - @Ibrahim99575Submitted over 1 year ago@namborghiniiPosted over 1 year ago
Good job on coding this even without the box shadow. Please pay attention to the text color next time! :)
0 - @nzcastilanSubmitted over 1 year ago
Had a difficulty of changing the image from mobile view to desktop view. I first tried to use srcset but can't seem to make it work so I ended up creating a div with a background image instead.
Not sure also what is the best practice to use in this kind of scenarios. any feedback would be helpful :) Thanks!
@namborghiniiPosted over 1 year agoLooks good so far! :)
Please add a hover effect for the button and try to center the text and icon within the button field next time.
Marked as helpful1