Submitted over 2 years ago
repsonsibe web design, mobile first, css grid, css flexbox
@Tucznik2
Design comparison
SolutionDesign
Solution retrospective
Hi guys, I've just completed this challenge and I would like some advice about things to improve.
Community feedback
- @Harshi786Posted over 2 years ago
Hey @Tucznik2
Congrats on completing the challenge.
Here are some tips-
-
For background image, use
body{background-size:contain;}
-
You can add a nice shadow effect on payment button.
.payment-button{ box-shadow: 0 20px 20px 0 rgb(56 42 225 / 19%); }
-
For accessibility issues:
- Wrap the whole code in
<main>
tag.<main class="wrapper"> //...... .......// </main> ```
- Take
- Wrap the whole code in
<footer> <div class="attribution"> Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank" >Frontend Mentor</a >. Coded by <a href="https://github.com/Tucznik2">Filip Kula</a>. </div> </footer>
If it really helped you then please mark it helpful. Thanks
Marked as helpful0 -
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord