All comments

• 

  Ayanwumi Abdulroheem Tunde•30

  @ayanwumi101Submitted over 2 years ago

  I had a tough time when I was implementing Redux-toolkit because it was first time using it.

  I'm sure of all the features and functionalities, they're all working perfectly.

  

  An e-commerce website built with React, CSS and Redux-toolkit

  1

  10
  

  Manik Kapoor•370

  @Manik2375Posted over 2 years ago

  Hi there, sorry to say but you have too many accessibility and HTML issues.

  About bit accessibility, you don't have any hover states or any other way of telling if you have clicking/hovering the button or not. And images are also bit of low resolution :sweat_smile:

  But I really like how you extended the challenge :thumbsup:

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Ben•770

  @BenConfigSubmitted over 2 years ago

  I put a lot of effort into the animations. Please tell me what you think and advise how I can improve my code. Many thanks.

  

  Static Job Listings (HTML, CSS, JS, jQuery)

  1

  30
  

  Manik Kapoor•370

  @Manik2375Posted over 2 years ago

  Hi @Ben nice work on animation, I really liked the hiding one, though talking about methodology, I see you are using BEM, but you aren't following it correctly :(

  According to it, element should not contain element, which in your case is .job__description which represents an element contains other elements.

  Nice work btw :thumbs_up:

  0

  This comment has received 2 repliesShow
• 

  Arbaz Yousaf•250

  @arbaz93Submitted over 2 years ago

  It was my first time creating this type of page. Any guidance would be helpful.

  

  ecommerce landing page

  1

  20
  

  Manik Kapoor•370

  @Manik2375Posted over 2 years ago

  Hi, to the point here are some things that I'd like to suggest:

  • The current product quantity can go below 0, which looks weird.
  • Take a look at html-accessibility issues.
  • The product page get connected with very bottom, I'd suggest to use some margins/padding
  • The backdrop of product images, when scrolling stays at top, perhaps use position: fixed for it.
  • Hover interaction of small thumbnail images, change cursor there.
  • They also don't have arrows to change image, which according to design you should have those.
  • Cart don't show how many items/or if items are selected or not.

  These are what I currently saw, good luck for your next project!

  0

  This comment has received 1 replyShow
• 

  Manik Kapoor•370

  @Manik2375Submitted almost 3 years ago

  

  Vanilla JS IP Address Tracker

  1

  10
  

  Manik Kapoor•370

  @Manik2375Posted almost 3 years ago

  Any suggestions ^^?

  0
• 

  Manik Kapoor•370

  @Manik2375Submitted almost 3 years ago

  No questions, of course any criticism?

  

  URL-shortening-API-landing-page with Everything covered in vanilla.

  2

  40
  

  Manik Kapoor•370

  @Manik2375Posted almost 3 years ago

  About issues, well I have to take a11y course for it. And about buttons one, didn't wanted to use js + for section one, headings aren't the direct child so..

  Ik design isn't much same, in terms of elements size, that's because this challenge was done just for own practice, nothing special. :P

  0