Responsive, No Frameworks, Mobile First, VanillaJS

Community feedback

• 

  Ren Ras•420

  @renrasPosted almost 3 years ago

  Your trash button has a child element in it which is targetted instead. Try using e.currentTarget instead because it is attached to the one with the event listener. But still it wouldn't work because the event is firing from the document which you have another event listener attached to which also gets fired. Instead of adding another event listener of the same type to the same element just merge them so It'll be easier to debug.

  0
• 

  Syed Ali Mansoor•460

  @syedalimansoorPosted almost 3 years ago

  Hello @titocs!

  Your solution looks great and works well. One thing I would recommend you to work on is animations and microinteractions. They can really improve the quality of your website. If you wish, you can check out my solution to this problem and tinker with the interactable elements to see what I mean!

  0

  ---

  

  Hfn•340

  @titocsPosted almost 3 years ago

  @alimansoor-create ahh i see, thank you. i will learn from your submission

  0
• 

  Gürkan Özer•420

  @gurkanozerPosted almost 3 years ago

  Trash button is working but its hard to click .

  0

  ---

  

  Hfn•340

  @titocsPosted almost 3 years ago

  @gurkanozer yes, must be click on its padding not on the logo. i dont know why

  0