Skip to content
OverviewSolutions
41
Comments
4

XIII

@minibruspPhilippines790 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

I’m currently learning...

NextJS

Latest solutions

View more solutions

Latest comments

  • XIII•790
    @minibrusp

    Nice Work Man. The solution is awesome. I love how the landing, signup and login looks. And also nice touch with using the firebase as a backend. I'll be looking forward with more ideas in the future.

  • XIII•790
    @minibrusp

    hi congratulations on completing this challenge. this is great.

    • there are lot of ways to achieve smooth transitions on modal since your using sass i would suggest to use css keyframes and animation. you can refer to this codepen for example but this codepen is pure core HtmlCssJs.

    codepen link

    • also i notice that opening the reward modal/back modal you can only select a certain reward by clicking on the label or the radio button. I think it would be nice if i can click the whole card to select the certain reward.

    all in all its a great solution. keep coding meow ^_^

    Marked as helpful
  • XIII•790
    @minibrusp

    overall this project is great.

    • I like the design where the math equation and the result are both visible
    • I like the way where i can input 1 + 3 then pressing another math symbol will calculate the previous equation.
    • I also like the part where i would input 1 + then after that i would press another math symbol for ex. (x) would change the previous symbol making the screen 1 x

    *Bug that i accidentally found

    • input 10 + 10 then press (+) or any math symbol then it will calculate then input the number for ex. 20 making the screen 20 + 20 then press delete button. It will not let me delete the newly inputted number.
    Marked as helpful
  • XIII•790
    @minibrusp

    there are two options i can give to you

    • adding these styles to p.saleprice class

    display: flex; justify-content: flex-start; align-items: center; gap: 1rem;

    • restructuring the html for div.price making both p.saleprice and span.oldprice as siblings then adding the above styles to div.price
    Marked as helpful