@elidrissidevSubmitted about 3 years ago
Form validation took me awhile. Feedback is welcome.
Hafizan Adli
@hafizanadliAll comments
- @hafizanadliPosted about 3 years ago
Almost similar to design, responsive, and feedback feature working well.
Keep up the great work!
1 - @mohdkabir18Submitted about 3 years ago
Feedbacks are Welcomed!!
@hafizanadliPosted about 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 - @Ryusuke66Submitted about 3 years ago
Any feedback is appreciated.
@hafizanadliPosted about 3 years agoWhat a great work!
Almost similar to design and responsive in every screen.
Keep up the great work!
1 - @salahudheen007Submitted about 3 years ago
any suggestions to improve my code?
@hafizanadliPosted about 3 years agoAppreciate 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 helpful0 - @renatomarquescoSubmitted about 3 years ago
any feedback is appreciated.
Thanks
@hafizanadliPosted about 3 years agoWhat 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 - @FahimMahmudJoySubmitted about 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.
@hafizanadliPosted about 3 years agoLooks 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 - @dan-dyeSubmitted about 3 years ago
Feedback appreciated!
@hafizanadliPosted about 3 years agoAppreciate 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 helpful1 - @Bharath-ShetSubmitted about 3 years ago
appreciate your feedback
@hafizanadliPosted about 3 years agoGreat 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 helpful0 - @rsantanacatalanSubmitted about 3 years ago
Hello Community
This my first app with React, I would appreciate your feedback. I enjoyed building the solution :)
@hafizanadliPosted about 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 - @rajulkoshtaSubmitted about 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.......
@hafizanadliPosted about 3 years agoAppreciate your effort, you can check my solution to give you reference of making responsive design
Keep learning and improving!
Marked as helpful0 - @ubed90Submitted about 3 years ago
Any Feedbacks Accepted 🚀🚀
@hafizanadliPosted about 3 years agoGreat, almost close to design, and responsive in any screen Keep up the good work!
0 - @VinayakDhamnekarSubmitted about 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 ?@hafizanadliPosted about 3 years agoYou can use background-size:contain instead of cover. And add background-color too
0
