Design comparison
Community feedback
- @timilehin223Posted over 1 year ago
There are a few issues that need to be addressed to meet the requirements specified in the style-guide.md file. The required font for the project is missing, and it needs to be imported from Google Fonts in the CSS file.
Furthermore, there is a discrepancy in the text color for the .text class, which needs to be corrected to match the style guide.
Regarding the background color, the body should be blue, but the .outerdiv class has been assigned this color instead. To adhere to the style guide, the background color should be applied to the body element.
Moreover, it appears that the current structure involves an unnecessary .outerdiv element, which contains the container for the card that holds the QR code and text. Since there is only one container involved, it would be simpler to place the card directly inside the container element, removing the .outerdiv.
Finally, it's recommended to consider increasing the width of the container element to ensure proper alignment and layout for the card.
To rectify these issues and achieve compliance with the style guide, follow these steps:
- Check the style-guide.md file to identify the required font.
- Visit the Google Fonts webpage and import the font into the CSS file.
- Correct the text color for the .text class to match the style guide.
- Apply the blue background color to the body element.
- Remove the unnecessary .outerdiv element and place the card directly inside the container element.
- Adjust the width of the container element to ensure an appropriate layout.
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