@alexiscenriquez
Latest comments
- @Jimender
Hello @alexiscenriquez, Nice Attempt on this Challenge.
Tips on how you can improve it:
- Do not give your card a fixed height. you can envelop your main html inside a container and give it height of 100vh.
- Place your text content and buttons inside a wrapper or container and apply padding to it.
- Your card have too much border radius reduce it to match the header img. Instead of % use pixels for border-radius.
- For background svg try
background: top/100% no-repeat url(../images/pattern-background-mobile.svg); background-color: var(--paleBlue);Marked as helpful - @TejasShekar@Jimender
Hi @TejasShekar, first of all Nice Work so far.
Tips on Improving:
- for box-shadow use
box-shadow: 0 1.5rem 2rem rgba(0,0,0,0.1);and try to tweak it until you are satisfied with the result. - wrap your container in main tag and use flexbox on your container.
- you forgot to add hover & active effects on buttons.
- you can wrap text content inside a div element and add padding so that it looks same through out the project
Sorry couldn't provide solution regarding svg.
Happy Coding :)
Marked as helpful - for box-shadow use
- @Gouse-saif@Jimender
- you should use appropriate colors.
body : Light-Gray: hsl(204, 43%, 93%); .priceList1: white;- In your why us section use ul element.
Nice Work. Happy Coding :)
- @muhammadshajjar@Jimender
Nicely done but i think you forgot to add box-shadow on your profile card. Otherwise, looks perfect to me :) .
Marked as helpful - @ptai66@Jimender
if you want to fix the position of buttons you can try absolute positioning.
