All comments

• 

  Franco Bernardi•150

  @francobernardiiSubmitted over 2 years ago

  How can my code be more efficient?

  Am I using Sass in the correct way?

  I don't feel comfortable with the way that i wrote my JS code, how can I do it cleaner?

  

  Responsive credit card challenge. Use of HTML, CSS, JS and SASS

  1

  20
  

  Byte•260

  @YonutDevPosted over 2 years ago

  Hello! 👋

  The card logo doesn't load. Modify img tag source location to assets/images/card-logo.svg from logo-container div.

  You use SCSS quite well excepting for some things.

  Personally, I understand the code written by you in JavaScript without more code comments, but I still recommend jQuery, a awesome tool. Try to work as efficiently and simply as possible. By separating everything and adding comments, you can be as well organized as possible.

  Look at this and practice it!

  Marked as helpful

  2

  This comment has received 1 replyShow
• 

  Felipe Alencar Giacomini•40

  @felipe-alencar6Submitted over 2 years ago

  

  Angular made solution

  1

  10
  

  Byte•260

  @YonutDevPosted over 2 years ago

  Hi there! 👋

  In my opinion, you can add some hover animations on rating buttons, try to get as close as possible to the design. One more thing, you can select multiple rating buttons. You can do something so that the user can select only one rating button and only if he selected one can press Submit.

  You could also look a little at the code to improve it & solve some of reported HTML Issues.

  Good luck!

  Marked as helpful

  1
• 

  Petru Banceanu•110

  @Petru14Submitted over 2 years ago

  The most difficult part about this project was the hover state, because a didn't used in the past week's and I lost a while till I figure out the solutions. Any kind of feedback is more then welcome.

  

  Responsive layout using HTML, CSS.

  2

  30
  

  Byte•260

  @YonutDevPosted over 2 years ago

  Hi there! 👋

  Congratulations! Everything you've done is almost done.

  In my opinion, what should be changed is the color of the border to a darker one, and still you could add some transitions to make it look a little more good.

  Read more about transitions

  Marked as helpful

  1
• 

  Reyhan Dilek•200

  @Reyhandilek07Submitted over 2 years ago

  İt was so easy and enjoyable. Thanks a lot!

  

  Responsive Huddle landing page using display flex

  1

  20
  

  Byte•260

  @YonutDevPosted over 2 years ago

  Hi there! 👋

  Congratulations! Everything you've done is almost done.

  I recommend that you leave some space between the social media links and the end of the page for screens that are not very big. You can use some <br> tags after social-medias div or using padding-bottom in the CSS of social-medias div. Besides this, you could also make the color of the button change faster, reducing the transition period, you could also add that when you hover over one of the social medias, its color changes, as in the design.

  Otherwise, everything seems fine.

  Marked as helpful

  0