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

  • Rickyβ€’ 460

    @rickyxyz

    Posted

    Cool React code.

    For the CSS, I usually sees people use rem instead of px for sizing. This article might be helpful PX or REM in CSS? Just Use REM

    Cheers 🍻

    Marked as helpful

    0
  • Cyril Salamiteβ€’ 70

    @Cyrilange

    Submitted

    I build this project mostly with flex , and I start the media query in 390px

    I am new to learning development , please feel free to be honest so I can progress

    My learning are javascript with react , html and Css ( with scss and bootstrap)

    Rickyβ€’ 460

    @rickyxyz

    Posted

    Fellow beginner here, so take this with a grain of salt πŸ§‚.

    I think this could be helpful for you BEM naming convention, since CSS names can get confusing quickly.

    Cheers 🍻

    Marked as helpful

    2
  • Renukta-21β€’ 130

    @Renukta-21

    Submitted

    What did you find difficult while building the project? It'll be cool if you guys could tell me what's weird of my solution, or maybe could giveme suggestions for best practices, thanks :)

    Rickyβ€’ 460

    @rickyxyz

    Posted

    I'm also a beginner, so take this with a grain of salt πŸ§‚.

    I think your HTML structure is odd.

    Take for example your 3 botCard cards, they have no container div to unify them. This could a problem when you have more than 3 cards, for example you would need to re-style the grid layout if there are 4 cards. I think it would be better if the botCard cards are contained in a div with grid or flex layout.

    Also you might want to look into semantic HTML5 guide.

    Cheers 🍻

    1
  • Rickyβ€’ 460

    @rickyxyz

    Posted

    Fellow beginner here.

    I think this could be helpful px or rem in css, since I usually see a lot of people use rem instead of px.

    Cheers 🍻

    Marked as helpful

    0
  • Talha Asreβ€’ 10

    @talhaasre

    Submitted

    I didn't face any difficulties during this project. I normally use em values is css but here I've used static px values so that might be a problem sometime in much lower width devices. Yes, I would like to know best possible solution for this challenge.

    Rickyβ€’ 460

    @rickyxyz

    Posted

    I'm a beginner myself, so you might want to take this with a grain of salt πŸ§‚.

    Maybe you could try making responsive font size and styling the elements using rem based on the responsive font size. Also, just as a suggestion, you might want to look at the most common screen size being used right now, so you can determine how small or how large of a screen you should accomodate.

    Cheers 🍻.

    0
  • Rickyβ€’ 460

    @rickyxyz

    Posted

    I'm also a beginner, so you might want to take this with a grain of salt πŸ§‚.

    Just a suggestion, I think this article could be helpful BEM naming convention.

    Cheers 🍻.

    Marked as helpful

    1
  • @anusha-fatima

    Submitted

    It's a very tricky project for me as a beginner but i try my best hope you all like it. Waiting for yours review and comments hope you find out your accurate current Age.

    Rickyβ€’ 460

    @rickyxyz

    Posted

    Just a small suggestion for the number input, maybe hide the default up and down arrow in the input. Since in firefox the arrow is always visible.

    This could be helpful. W3S Hide Arrows From Input Number

    Cheers 🍻

    Marked as helpful

    1