Skip to content
OverviewSolutions
15
Comments
8

Felipe Stefani

@felipestefani330 points

I'm just doing what I love! Sports, coding, going out with my family and friends, drinking some wine once a while and living to make the difference.

I’m currently learning...

Nowadays I'm practicing Vue to fix knowledge and TS as well. I'm already have background in react so I still doing codes in both to improve my skills!

Latest solutions

View more solutions

Latest comments

  • Felipe Stefani•330
    @felipestefaniPosted over 1 year ago

    I forgot the mobile menu!

    Working on that yet!

    0
  • Felipe Stefani•330
    @felipestefaniPosted over 1 year ago

    Hey, Mahmoud!!

    Your designs are always great! I've been following you since a while and it's very good to see how your projects are.

    In this one, the only observation to do is in your logic. When selecting the additional, whether selected or not, the total includes all of them. I didn't check the code at all, but thinking about it it's a simple thing to correct.

    Beside that, as I've already said, I love to see the results you show to us ☺️

    1
  • Felipe Stefani•330
    @felipestefaniPosted over 1 year ago

    Hey, @BrunnoHibbeln!! How you doing? Hope great 😊

    Your project is very nice. Really!

    Don't think about errors, but thinks about you can improve!

    I've noticed 2 things: The font-size, for desktop, is too little, and it can cause some user discomfort. At the summary, the itens don't have a background.

    In my opinion, both things are only details that with practice you'll recognize easily.

    Besides that, I think you have a great project in hands! Congrats 🎉

    Marked as helpful
    0
  • Felipe Stefani•330
    @felipestefaniPosted over 1 year ago

    Hey, Joash!

    How are you? Hope doing great! 😉

    I liked a lot the hover status you have done in the button! 👌

    Beside that, you really could have used a href to navigate to the page, but it is not as performative as using js.

    and you could have used js to do some validations as email verification. A good idea is implement a regex to support the validations:

    mailFormat = /^\w+([\.-]?\w+)*@\w+([\.-]?\w+)*(\.\w{2,3})+$/

    This way your user experience get more interesting!!

    When I come to see the preview site, the images are broken... Anything's all right with deployment?

    Keep going, your project ir very nice! I liked a lot 💫

    1
  • Felipe Stefani•330
    @felipestefaniPosted over 1 year ago

    Despite the css gap, the project get very nice!

    All the logic seems to be perfect and the classes are bind dynamically, as I could see.

    I don’t know if you know tailwind or bootstrap, but it helps a lot with the style.

    I’kk let the links here to check it out: tailwind bootstrap

    In my projects I’m coding all css as well, but now it’s my focus. If the idea is to code the project, this tools helps a lot!

    Great job! I liked a lot 😉

    Marked as helpful
    0
  • Felipe Stefani•330
    @felipestefaniPosted over 1 year ago

    Hey! How are you? Hope doing great!!

    I'd liked very much your desktop design. Congrats 🎉

    Just one observation: you could made your design responsive for mobile as well.

    In your css you could use media to solve many of your problems in this kind of issue

    @media (max-width: 700px) and (min-width: 375px) {

}

    The example above is just an example of how could you solve this problem.

    You can change the widths and the min /max position, depending on how you start your design (mobile or desktop first).

    Despite that, and backing to the project done, you did very good job!!!! 👏👏👏👏

    0
View more comments