@sarfarazstark
@MesrouaDjamel
Submitted
What are you most proud of, and what would you do differently next time?
now I use figma before code
What challenges did you encounter, and how did you overcome them?
I've never deployed the application before, but reading the documentation on vercel makes it easier.
@sarfarazstark
Posted
Hello @MesrouaDjamel,
Your solution looks great! For an even closer match to the given design, consider the following adjustments:
max-width of your main tag to 300px.0.75rem left and right padding to your p tag.These changes will help your solution align more closely with the provided design.
Best regards, Sarfaraz
Marked as helpful
@amiralamf4
Submitted
@sarfarazstark
Posted
Solution
set the body and main div height to auto
add body padding top and bottom to 2rem
add gap of 10px to socialmedia tag
Marked as helpful
@amiralamf4
Submitted
@sarfarazstark
Posted
The issue is with your width and height dont set explicit height and width also always use max-width for this kind of component
for now use this solution:
max-width: 870px;
max-height: 350px;
Marked as helpful
@rabin-thami
Submitted
@sarfarazstark
Posted
Use color from style-guide.md for text
@Mahmoud974
Submitted
Here's my project multi form, as usual, i used typescript and tailwindcss, comment please 😊
@sarfarazstark
Posted
In the input field add some paddings
Even though I filled in all those input fields your code still shows an error try Debug this
I think you validating the field using the input event use click event to check input fields
@Ilxomjon26
Submitted
hammaga salom . bu mening yangi proektim . Korib baho bering , hato kamchiliklarini yozib qoldiring . hammaga omad
@sarfarazstark
Posted
I have some suggestions for you:
👉 Use main and section tag
👉 Reset the default style of the website with this
*{ box-sizing: border-box; margin: 0;} in your style sheet
👉 Instead of giving random class names give them some descriptive name
👉 Use font style which is included in style-guide.md
👉 Don't use px instead use rem em
👉 There is a scrollbar why please try to fit everything that it will not give the scrollbar
👉 I see that you just make a photo like a website as it has no text field nor button tag please use them to make a newsletter email box
I hope my suggestion helps you 😊
@pkkppk
Submitted
How to change the image when the screen size is reduced without using js?
@sarfarazstark
Posted
use display: none; with media queries
@LegendCoder20
Submitted
Please Give an Response on My Project. THANKYOU.
@sarfarazstark
Posted
Great work,
But for making it more accurate you should change the color of your headings with this
color: hsl(218, 44%, 22%);
also, change the tag to h1Marked as helpful
@coderwww
Submitted
Any comments and suggestions are welcome!
@sarfarazstark
Posted
Great Work.
You should add shadow to it because it's missing in it
.order-summary{ box-shadow: 0px 12px 21px -9px rgba(0,0,0,0.75); }
Marked as helpful
@modersetech
Submitted
@sarfarazstark
Posted
Everythings look great just remove that footer text so it look more accurate
Marked as helpful
@modersetech
Submitted
@sarfarazstark
Posted
Great work
Marked as helpful
@ali007-depug
Submitted
Give me some advice?
@sarfarazstark
Posted
If you don't have knowledge in CSS or just started leaning please check freecodecamp it will help you
Marked as helpful
