Please provide any feedback. Thanks in advance.
Md Ajmal Shadab
@mdajmalshadab
All comments
- Md Ajmal Shadab• 410
@mdajmalshadab
Posted
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 helpful
0 - Sean Brookstein• 150
@CassiuSRSA
Submitted
Any feedback will be greatly appreciated :D
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey 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 helpful
1 - Josh• 20
@J0shcodes
Submitted
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.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
hey you have done a great job, I liked the responsiveness of the page. Go head and take a new challenge. Keep coding!
0 - Nabil• 160
@Nabil19911
Submitted
Hey fellow developers, please review and give feedback. Thank you in advance.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey, 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 helpful
0 - Andrei Minciuna• 40
@AndreiM987
Submitted
I couldn't solve the background problem with the two circles. Any idea how can i solve the problem?
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey 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 - Ezra Adeyinka• 40
@adeyinkaezra123
Submitted
Your feedback is very much appreciated
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey 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 helpful
0 - Alan Santos• 50
@alan-ssantos
Submitted
I have no questions yet 😅, but any feedback would be very appreciated. 🥰
Md Ajmal Shadab• 410@mdajmalshadab
Posted
You have done it perfectly! I liked it quite much. Keep doing new challenges 😊
0 - ankit sahu• 80
@nerdgeek007
Submitted
Feedback and Improvement are highly appreciated, so kindly give your feedback.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Well 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 - SakshiSawant• 50
@SakshiSawant
Submitted
Suggestions are needed to improve accessibility issues
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey 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 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
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey 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 helpful
1- John Carson• 50
@wppaing
Submitted
I'm just a newbie. Feedbacks welcome and thanks you all💚
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey, 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 - SakshiSawant• 50
@SakshiSawant
Submitted
I am not able to add multiple background images so can anyone please give any solution how to do that
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Another 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 - SakshiSawant• 50
@SakshiSawant
Submitted
I am not able to add multiple background images so can anyone please give any solution how to do that
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey, nice work for the card section, for background part you can use the following CSS property in you main/body element
background-image: url("image-url-1"), url("image-url-2");
and then accordingly position your background usingbackground-position: left -80px top 50px, left -570px top -300px;
NOTE: These positions are random and just for example, you can position your two background images by separating it with comma.Marked as helpful
1 Hello guys. Let me know what you guys think about this webpage. Any feedback to make this better is appreciated. Thanks in advance!
P.S.: I would like to know how to add color to an image i.e. needed here for this challenge. I added filter with opacity. It did get the job done but it doesn't look exactly as shown in the design.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
That's very good work! keep doing new challenges :)
Marked as helpful
1- Pancha• 30
@Pancha99x
Submitted
This is my first challenge as a Newbie. Any kind of suggestion will be kindly accepted. Thank you.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey, really good work for the first challenge. Keep coding!
Marked as helpful
0 - Riz• 30
@vantevie
Submitted
- Not quite sure if I used the correct color for the image overlay.
- Almost positive the image to text ratio on the Desktop view is incorrect. Would be glad to get any help for this too 🐱👓😄
Any feedback would be appreciated!
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Nice Work! Try using
display: flex
in the parent container of your image and card body, it'll automatically divide equal space for your image and card body.0 - Heath Allred• 35
@Heathallred4
Submitted
This is my first project with no tutorial or anything. Only the Google Gods to get me through. Any feedback is appreciated.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey, that's really good one. I think you just forgot to change the color of headings and paragraphs, change it to dark blue and it'll be same as original template. Keep Coding!
Marked as helpful
1 - Erel Ropeta• 385
@erelropeta
Submitted
I tried making the two circles in the background as responsive as I can, but I think there are breakpoints. I would love to hear feedbacks to improve on this :)
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hey Great Work! Regarding those breakpoints I would recommend using
background-image
CSS property inside yourmain
container, and then accordingly position your background usingbackground-position
property to match the desired layout.0 - jhuanjulio• 230
@jhuanjulio
Submitted
My first project using Javascript :) Feel free to point out some pointers. Thankyou!
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Hi! nice work , just wanted to add up that you should use click
eventListener
over your entire FAQ question rather than just the arrow part. That would be more user friendly.Marked as helpful
0 - hidayahmsabah• 160
@hidayahmsabah
Submitted
-
I couldn't figure out how to change the svg bg in mobile vs desktop, so resorted to using CSS transform of the mobile background to better fit the desktop version.
-
I tested the page on mobile, while trying to close the accordion for a question, the question's text changes color to pseudo-element hover which is not what I wanted. When used on desktop it works completely fine.
If anyone can assist me with these or other issues that I might have missed, feel free to point them out. Thanks a bunch!
Md Ajmal Shadab• 410@mdajmalshadab
Posted
I loved the responsive transition of your page (the moment one switches to mobile screen). Although just use the
overflow: hidden
in the main card class of the image so that it won't exceed the card boundary.Marked as helpful
0 -
- Marvin Ngugi• 200
@phoenix-mkay
Submitted
Any feedback and suggestions on how I can improve are very welcome!
Md Ajmal Shadab• 410@mdajmalshadab
Posted
That's some good work! I liked the responsiveness of the page. Just use right padding inside every card to achieve the exact same layout as shown in the screenshot.
Keep coding!🥳
Marked as helpful
0 - vasileios• 30
@vasilisalmpanis
Submitted
Any feedback and suggestions is welcome. Can anyone please explain how i can change the image color to violet? Thank you in advance.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Nice work! for the image filter: Use a div container for the image, set container background to violet color, then set image opacity to some value like 0.8, and in the end use below CSS property for your image mix-blend-mode: multiply;
0 - Shawn• 90
@Shawwnscott
Submitted
I wasn't able to make the background how it should be. Please help.
Md Ajmal Shadab• 410@mdajmalshadab
Posted
Use a div container for the image, set container background to violet color, then set image opacity to some value like 0.8, and in the end use below CSS property for your image mix-blend-mode: multiply;
Marked as helpful
0