Skip to content
OverviewSolutions
12
Comments
113
Hafizan Adli
@hafizanadli

All comments

  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    Almost similar to design, responsive, and feedback feature working well.

    Keep up the great work!

    1
  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago
    • for centering the component you can set body minimum height to 100vh and center the row-feature element with flex in body.
    • Adjust the border radius when in mobile view

    Keep up the good work!

    1
  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    What a great work!

    Almost similar to design and responsive in every screen.

    Keep up the great work!

    1
  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    Appreciate your effort!

    My suggestion is instead of using absolute px value for setting component height or width, it'll be better to use relative unit values such as %,rem,em,vh,vw,etc. Easier for building responsive component

    Keep learning and improving!

    Marked as helpful
    0
  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    What a great work!

    Your site looks good, very close to design and responsive in any screen

    The part you can improve is to use semantic html.

    Keep up the good work!

    0
  • Fahim Mahmud•165
    @FahimMahmudJoySubmitted over 3 years ago

    Any suggestion on responsiveness on mobile devices is highly appreciated. I can make my site look okay on desktop and laptop but struggle with small screens. Thanks.

    HTML and CSS site using flexbox

    1
    Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    Looks good in desktop view. You should make another media queries styling for smaller viewport. Reduce mergin value when in mobile view, the current margin is too big for mobile view.

    Keep up the great work!

    1
  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    Appreciate your great effort!

    My suggestion is try to use relative unit value such as %,rem,em,vw,vh,etc. instead of using absolute px value. It will be easier for building responsive component.

    Other than that, your component almost close to design and responsive enough.

    Keep up the great work!

    Marked as helpful
    1
  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    Great work!

    You already did good on how to centering the component. but you 10% margin in container is too big when in big screen. you can change it to margin:10px or 1rem maybe, it will make the component looks better when in large screen.

    Keep up the good work!

    Marked as helpful
    0
  • Rodrigo•20
    @rsantanacatalanSubmitted over 3 years ago

    Hello Community

    This my first app with React, I would appreciate your feedback. I enjoyed building the solution :)

    Tip Calculator App

    3
    Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago
    • For better ux, in first load, when user focus on the input (bill or number of people) and type number it will be better if they do not have to delete the current value first. You could give it a placeholder instead of give initial value.
    • You can hide arrows button for number input, it looks cleaner.

    other than that, the functionality working well and responsive. Keep up the great work!

    0
  • Rajul Koshta•10
    @rajulkoshtaSubmitted over 3 years ago

    I faces so much problem in making it responsive for mobile an someone tell a better technique to tackle responsiveness and also tell me bit more about that mobile first procedure. I am bit of confused because I have always tried desktop first or some good tips to start with . thank you, please provide feedback , its worth if you.......

    HTML CSS BOOTSTRAP

    1
    Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    Appreciate your effort, you can check my solution to give you reference of making responsive design

    Keep learning and improving!

    Marked as helpful
    0
  • Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    Great, almost close to design, and responsive in any screen Keep up the good work!

    0
  • VInayak D•20
    @VinayakDhamnekarSubmitted over 3 years ago

    Hi everyone, I am beginner in web development and this is the first web development project I have worked on. I am hoping to get some feedback and tips on my code and project approach. What are the areas that can be improved/ tackled with different approach.?

    I struggled to adjust the background image as shown in mock up. I used following code but could not achieve the same result. body{ background-image: url(); background-size: contain; background-repeat: no-repeat; min-height: 100vh; How to adjust the background image ?

    Css flexbox

    2
    Hafizan Adli•1,190
    @hafizanadliPosted over 3 years ago

    You can use background-size:contain instead of cover. And add background-color too

    0