Skip to content
OverviewSolutions
27
Comments
7
Ricky
@rickyxyz

All comments

  • Rickyā€¢460
    @rickyxyzPosted 12 months ago

    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
    @CyrilangeSubmitted 12 months ago

    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)

    CSS flex and media query ( 390px )

    3
    Rickyā€¢460
    @rickyxyzPosted 12 months ago

    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-21Submitted 12 months ago

    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 :)

    Responsive Landing Page with grid

    1
    Rickyā€¢460
    @rickyxyzPosted 12 months ago

    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
    @rickyxyzPosted 12 months ago

    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
    @talhaasreSubmitted 12 months ago

    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.

    Responsive QR code component using html & CSS.

    2
    Rickyā€¢460
    @rickyxyzPosted 12 months ago

    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
    @rickyxyzPosted 12 months ago

    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 Aliā€¢100
    @anusha-fatimaSubmitted 12 months ago

    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.

    age-calculator-app-main.

    1
    Rickyā€¢460
    @rickyxyzPosted 12 months ago

    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