Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • P

    @ikitamalarose

    Submitted

    What are you most proud of, and what would you do differently next time?

    I am proud to have able to carry out this project already, which was a little difficult at first to organise my ideas and the structure of my html in order to manage it well with JavaScript.

    What challenges did you encounter, and how did you overcome them?

    On the management of the buttons and the calculation it was a little complicated at first,but after taking the time to organise my html code well. It was good.

    What specific areas of your project would you like help with?

    So i have a problem with the choice of buttons. I explain myself when the user clicks on the button choices we can see the value in the custom field. But in the project the choice of the button must not be displayed in the custom field. If anyone would like to enlighten me on the right process, thank you.

    duongns-vn 210

    @duongns-vn

    Posted

    The code is very good and the application works perfectly. Congratulation.

    0
  • @AbdullahSoulat

    Submitted

    What are you most proud of, and what would you do differently next time?

    Managing data from a json file and displaying it properly on the page.

    What challenges did you encounter, and how did you overcome them?

    Some issues with checking classes in JavaScript

    What specific areas of your project would you like help with?

    Currently, I remove 6 cards, when updating the UI. I would like to remove all cards in the grid except the profile using classes

    duongns-vn 210

    @duongns-vn

    Posted

    Very good code, congratulations. But perhaps the Zoom (@keyframes pop) effect seems to cause a not-so-great experience on mobile. Wish you happy coding.

    0
  • duongns-vn 210

    @duongns-vn

    Posted

    Very good, I really like the way you did it. Have fun coding

    1
  • duongns-vn 210

    @duongns-vn

    Posted

    Very good, the code is very neat and the click function is very effective. Have fun coding

    Marked as helpful

    0
  • duongns-vn 210

    @duongns-vn

    Posted

    The code is really neat and awesome. Well done my friend, have fun coding.

    0
  • @bossthekhani

    Submitted

    What are you most proud of, and what would you do differently next time?

    I am proud of CSS property flex and Grid.

    What challenges did you encounter, and how did you overcome them?

    I encountered many challenges while completing the task.

    What specific areas of your project would you like help with?

    .

    duongns-vn 210

    @duongns-vn

    Posted

    Well done my friend. I think you should use the max-width attribute for the cards so it doesn't zoom freely, then everything will be better. However that is just a small problem. Wishing you always have fun with coding

    0
  • duongns-vn 210

    @duongns-vn

    Posted

    Very good, my friend.

    Everything seems to work fine with the regular screen. However, with the vertical mobile viewport, the .product--card--image class has been escaped.

    You can use img:src tag on html file and add object-jit:cover property, instead of adding img as background. Along with that, use background and border-radius for .product--card.

    Wish you always have fun coding

    Marked as helpful

    1
  • @KomisarzRex

    Submitted

    What challenges did you encounter, and how did you overcome them?

    I had a problem with changing the photo depending on the resolution, but the picture tag helped me.

    What specific areas of your project would you like help with?

    I still have a problem making the website responsive.

    duongns-vn 210

    @duongns-vn

    Posted

    You can use flexbox-direction for .product-card , along with using object-fit:cover, object-postion: center for images, which will make operations faster and the code shorter. Try your best, my friend

    Marked as helpful

    1
  • duongns-vn 210

    @duongns-vn

    Posted

    It's so good, my friend, try your best.

    0
  • duongns-vn 210

    @duongns-vn

    Posted

    Not same, Using more Flexbox, the asset path is not correct, there is no hover style

    0