Mohamed ELIDRISSI• 455
@elidrissidev
Submitted
Form validation took me awhile. Feedback is welcome.
Hafizan Adli
@hafizanadli
All comments
- Hafizan Adli• 1,190
@hafizanadli
Posted
Almost similar to design, responsive, and feedback feature working well.
Keep up the great work!
1 @mohdkabir18
Submitted
Feedbacks are Welcomed!!
Hafizan Adli• 1,190@hafizanadli
Posted
- 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- Yassine Ghezal• 870
@Ryusuke66
Submitted
Any feedback is appreciated.
Hafizan Adli• 1,190@hafizanadli
Posted
What a great work!
Almost similar to design and responsive in every screen.
Keep up the great work!
1 - salahudheen k• 265
@salahudheen007
Submitted
any suggestions to improve my code?
Hafizan Adli• 1,190@hafizanadli
Posted
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 - Renato Marques• 130
@renatomarquesco
Submitted
any feedback is appreciated.
Thanks
Hafizan Adli• 1,190@hafizanadli
Posted
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
@FahimMahmudJoy
Submitted
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.
Hafizan Adli• 1,190@hafizanadli
Posted
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 - Daniel Dye• 25
@dan-dye
Submitted
Feedback appreciated!
Hafizan Adli• 1,190@hafizanadli
Posted
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 - Bharath Shet• 65
@Bharath-Shet
Submitted
appreciate your feedback
Hafizan Adli• 1,190@hafizanadli
Posted
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
@rsantanacatalan
Submitted
Hello Community
This my first app with React, I would appreciate your feedback. I enjoyed building the solution :)
Hafizan Adli• 1,190@hafizanadli
Posted
- 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
@rajulkoshta
Submitted
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.......
Hafizan Adli• 1,190@hafizanadli
Posted
Appreciate your effort, you can check my solution to give you reference of making responsive design
Keep learning and improving!
Marked as helpful
0 - UbedS• 320
@ubed90
Submitted
Any Feedbacks Accepted 🚀🚀
Hafizan Adli• 1,190@hafizanadli
Posted
Great, almost close to design, and responsive in any screen Keep up the good work!
0 - VInayak D• 20
@VinayakDhamnekar
Submitted
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 ?Hafizan Adli• 1,190@hafizanadli
Posted
You can use background-size:contain instead of cover. And add background-color too
0
