Skip to content
OverviewSolutions
31
Comments
11
grgrnkoo
@grgrnkoo

All comments

  • SabaMarghania1•530
    @SabaMarghania1
    What are you most proud of, and what would you do differently next time?

    This was good challenge. i really like new design files, they are good structured, and comfortable for work

    Mortgage Calculator

    #react
    1
    grgrnkoo•520
    @grgrnkoo

    Good solution! Glad to see you completed it!

    The only thing that doesn't fit the task is that you don't have focus states on your input. It should have a lime border and a lime background on the input decorations.

    Also, your input decorations are moving up on error states, add a bit of padding and overlay: hidden. Aaaaaand also if you want to complete it perfectly, add a cursor pointer on each interactive element:)

    Other than that, your work is perfect. Happy coding!

    Marked as helpful
  • Amjad Shadid•810
    @amjadsh97
    What are you most proud of, and what would you do differently next time?

    I learned how to handle product counts, unique product lists, and total price calculations in a shopping cart context using JavaScript. Here's a breakdown of what I implemented:

      // Calculate the count of each product
const productCounts = selectedProducts.reduce((acc, product) => {
	acc[product.name] = (acc[product.name] || 0) + 1;
	return acc;
}, {} as Record);

// Create a list of unique products with their counts
const uniqueProducts = selectedProducts.reduce((acc: Product[], product) => {
	if (!acc.find(p => p.name === product.name)) {
		acc.push(product);
	}
	return acc;
}, []);

// Calculate total price
const totalPrice = selectedProducts.reduce((acc, product) => acc + product.price, 0);



### What specific areas of your project would you like help with?
I want feedback on state management

    Product list with cart solution

    #react
    1
    grgrnkoo•520
    @grgrnkoo

    Good job! All nuances are perfectly worked out! Can not find anything that could be upgraded!

    Marked as helpful
  • grgrnkoo•520
    @grgrnkoo

    Nice job, but looks like you lost the closing " on each quote:)

  • grgrnkoo•520
    @grgrnkoo

    Great solution! I took your fetching method and tried to apply it to my solution! Thanks

    But I think you forgot to add interactivity for a dice button. You can try dice.addEventListener('click', randomAdvice) after initializing a dice value with getElementById

  • grgrnkoo•520
    @grgrnkoo

    Your solution is amazing! I just finised mine, opened your code and realised that I totally forgot to remove error class while clicking a Reset button

    Nice attention to the details. Good job!

  • Hamidqasemi90•30
    @Hamidqasemi90
    What are you most proud of, and what would you do differently next time?

    sorry the picture is not working.

    QR code using css html

    2
    grgrnkoo•520
    @grgrnkoo

    Hi! You can try adding a dot before image’s path. Like .images/image-qr-code.png instead of images/image-qr-code.png

    this should work

  • J. Júnior•130
    @jjuniorbrasil
    What are you most proud of, and what would you do differently next time?

    Proud of doing it using the 'state' logic.

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

    No specific challenges.

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

    None for now.

    Time tracking dashboard

    1
    grgrnkoo•520
    @grgrnkoo

    Damn! Your project looks solid. Just finished the same project and your solution with states is the thing that I want to overthink right now. Happy coding!

    UPD: Checked the media queries. You should spend a bit more time to apply all screen sizes. Text doesn't fit the cards size in the width range of 1030 - 770 px

    Marked as helpful
  • Willen Garcia•170
    @willengarcia
    What are you most proud of, and what would you do differently next time?

    entregaria o mais rápido possível, e estudaria regex.

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

    A utilização de Regex, e o uso de localStorage

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

    nenhuma

    regex para validação do email, localStorage para recuperação de email

    1
    grgrnkoo•520
    @grgrnkoo

    amazing job!

  • svo15•230
    @svo15
    What are you most proud of, and what would you do differently next time?

    im proud about not give upping and second time i will better study basics

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

    mobile version text problems in where was name im just change width to min-width

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

    object overflow

    css flex

    1
    grgrnkoo•520
    @grgrnkoo

    Nice solution! You probably should take a bit more time on adjusting styles to reach a look-a-like design from the challenge. But technically your JS code is amazing and much simpler than mine! Good job!

  • grgrnkoo•520
    @grgrnkoo

    Why do Button colors and h1 margin not apply in the solution card?! First time facing such an issue

  • grgrnkoo•520
    @grgrnkoo

    After reviewing other solutions I understood only one thing. It doesn't have to be full-flexible. I did a ton of extra job applying this. But it still works, so my job was not pointless