Please provide any feedback. Thanks in advance.
Md Ajmal Shadab
@mdajmalshadabAll comments
- @alias01Submitted about 3 years ago@mdajmalshadabPosted about 3 years ago
Your result looks pretty good, just try aligning all 3 buttons in your card. Rest is perfectly fine. Go ahead and take a new challenge.
Marked as helpful0 - @CassiuSRSASubmitted about 3 years ago
Any feedback will be greatly appreciated :D
@mdajmalshadabPosted about 3 years agoHey nice work, you just need to add some background color to your page body so that it could match with the challenge template. Keep coding!😊
Marked as helpful1 - @J0shcodesSubmitted about 3 years ago
Here is My Solution for the order summary. I want to find out what you guys think, because I feel I didn't do justice to the colors.
@mdajmalshadabPosted about 3 years agohey you have done a great job, I liked the responsiveness of the page. Go head and take a new challenge. Keep coding!
0 - @Nabil19911Submitted about 3 years ago
Hey fellow developers, please review and give feedback. Thank you in advance.
@mdajmalshadabPosted about 3 years agoHey, nice work, everything is fine except the fact that the card is jumping a bit while changing the screen size and try increasing the length of the button shadow and add hovering color change on the button if possible. Happy Coding!
Marked as helpful0 - @AndreiM987Submitted about 3 years ago
I couldn't solve the background problem with the two circles. Any idea how can i solve the problem?
@mdajmalshadabPosted about 3 years agoHey nice work on the card! For the background you can do....
Add following CSS property inside your page body/main
background-image: url("url-1"), url("url-2)
The two image urls separated by comma Next you can position it accordingly also seperated by comma for example....background-position: left 10px top 10px, left 30px top 30px;
These numerical values are just for example, tweek it as it best fits. Keep coding!
1 - @adeyinkaezra123Submitted about 3 years ago
Your feedback is very much appreciated
@mdajmalshadabPosted about 3 years agoHey you have done it nicely, only you just need to reduce some bottom padding of the card, then it'll match with the given challenge. Keep coding ✌️
Marked as helpful0 - @alan-ssantosSubmitted about 3 years ago
I have no questions yet 😅, but any feedback would be very appreciated. 🥰
@mdajmalshadabPosted about 3 years agoYou have done it perfectly! I liked it quite much. Keep doing new challenges 😊
0 - @nerdgeek007Submitted about 3 years ago
Feedback and Improvement are highly appreciated, so kindly give your feedback.
@mdajmalshadabPosted about 3 years agoWell I guess, you submitted your solution in wrong challenge, btw try changing the color of car names to white in your card. And also try aligning all three buttons on same level, rest is pretty good. Keep coding✌️
0 - @SakshiSawantSubmitted about 3 years ago
Suggestions are needed to improve accessibility issues
@mdajmalshadabPosted about 3 years agoHey nice work! Just reduce left/right padding inside each button and add little margin to the top and bottom of the button, rest is good I guess. Keep coding!
1 - @darkabysmSubmitted about 3 years ago
I could not leave the button at the bottom, and all three aligned. I think I did something wrong with the Flexbox but I would not know what exactly
@mdajmalshadabPosted about 3 years agoHey nice work, you need to change the color of the button in hovering state by using
:hover
with your button class in CSS. You can also add right padding to individual cards to match your design to the given challenge.Keep Coding!
Marked as helpful1 - @wppaingSubmitted about 3 years ago
I'm just a newbie. Feedbacks welcome and thanks you all💚
@mdajmalshadabPosted about 3 years agoHey, that's a very good work for the first challenge, one thing I would like to pin point here is that you did not used the background image (wavy curve) that is shown in the original design.
Keep coding! Keep doing new challenges!😊
0 - @SakshiSawantSubmitted about 3 years ago
I am not able to add multiple background images so can anyone please give any solution how to do that
@mdajmalshadabPosted about 3 years agoAnother way that you can use is what I did while doing this challenge.....
I created a parent class 'bg' and its two child classes 'bg-1` and 'bg-2', following are the CSS property I used for positioning my two background images.
.bg { margin: 0; display: flex; justify-content: center; position: relative; width: 100%; height: 100%; z-index: -1; overflow: hidden; }
.bg-img-1 { position: absolute; right: 52%; bottom: 40%; }
.bg-img-2 { position: absolute; left: 50%; top: 50%; }
if you have any doubt you can refer to my github code
1