Headbwoi• 1,100
@xeuxdev
Submitted
I would appreciate any kind of feedback ☺️
Abdur Rouf Tarek
@rouftarek
All comments
- Abdur Rouf Tarek• 120
@rouftarek
Posted
Nice work.
Couple of things...
- add the background image in
main .container { } - Center the .container, not .row
<div class="tail"> <a href="#">Proceed to Payment</a> <p>Cancel Order</p> </div>Here I think Proceed to Payment would be better with a button same for Cancle Order. Atleast it should be a link.
Marked as helpful
0 - add the background image in
- P
@guilhermehto
Submitted
Had a lot of fun on this one. Any feedback would be appreciated :)
Abdur Rouf Tarek• 120@rouftarek
Posted
Hi, nicely done.
One thing if I may, you should add a 'background-size: contain;' or 'background-size: cover;' in ./src/App.css file under #main-section { }
#main-section { height: 100vh; background-image: url('./assets/pattern-background-mobile.svg'); background-repeat: no-repeat; background-size: cover;' background-color: var(--pale-blue); display: flex; align-items: center; justify-content: center; }Marked as helpful
1 - Sonu Swapan Dutta• 530
@Sonu-Dutta
Submitted
Abdur Rouf Tarek• 120@rouftarek
Posted
Nice work. But I think you forgot the hover state.
Marked as helpful
0
