Ahsan Raza
@AhsanCommitsAll comments
- @WesSnoSubmitted over 1 year ago@AhsanCommitsPosted over 1 year ago
Dear @WesSno,
While reviewing your project, I noticed a couple of design issues that I wanted to highlight.
Firstly, it seems that the hover feature on the button, as specified in the challenge requirements, has not been implemented. š To improve the user experience, I recommend adding a hover effect to the button using CSS, such as changing the background color or adding a subtle shadow, to indicate interactivity. āØ
Additionally, I noticed that you have used
width: 100vw
on some elements. However, it's important to keep in mind that this property does not account for vertical scrollbars, resulting in a horizontal scrollbar appearing on your site. To ensure a better user experience, it's advisable to avoid usingwidth: 100vw
in your design.I appreciate your use of Flexbox and the effort put into the project. Adding the hover feature and addressing the width issue will enhance the overall usability of your website. šŖ
You can also visit my solution to this:
Solution URL: https://www.frontendmentor.io/solutions/responsive-single-price-grid-component-using-css-grid-and-nextjs-hH_SBJj-io Live Site URL: https://single-price-grid-component-one-cyan.vercel.app/
If you have any further questions or need assistance with implementing the hover feature or addressing the width issue, please feel free to reach out. Keep up the good work! š
Best regards, @AhsanCommits
Marked as helpful0 - @BennatBjoernSubmitted over 1 year ago@AhsanCommitsPosted over 1 year ago
Dear @BennatBjoern,
While reviewing your project, I noticed that the hover feature on the button, as specified in the challenge requirements, has not been implemented. š
To improve the user experience and fulfill the challenge criteria, I recommend adding a hover effect to the button. This can be achieved through CSS, such as changing the background color or adding a subtle shadow, to indicate interactivity. āØ
Overall, I appreciate your use of Flexbox and the effort put into the project. Adding the hover feature will enhance the visual feedback for users and make the call-to-action button more engaging. šŖ
You can also visit my solution to this:
Solution URL: https://www.frontendmentor.io/solutions/responsive-single-price-grid-component-using-css-grid-and-nextjs-hH_SBJj-io Live Site URL: https://single-price-grid-component-one-cyan.vercel.app
If you have any further questions or need assistance with implementing the hover feature, please feel free to reach out. Keep up the good work! š
Best regards, @AhsanCommits
Marked as helpful1 - @Zy8712Submitted over 1 year ago
Made using flex box. Any suggestions for improvement would be greatly appreciated. Thanks.
@AhsanCommitsPosted over 1 year agoDear @Zy8712,
While reviewing your project, I noticed that the hover feature on the button, as specified in the challenge requirements, has not been implemented. š
To improve the user experience and fulfill the challenge criteria, I recommend adding a hover effect to the button. This can be achieved through CSS, such as changing the background color or adding a subtle shadow, to indicate interactivity. āØ
Overall, I appreciate your use of Flexbox and the effort put into the project. Adding the hover feature will enhance the visual feedback for users and make the call-to-action button more engaging. šŖ
By the way, your GitHub profile readme file looks awesome! š May I know how you created it? It's impressive!
You can also visit my solution to this:
Solution URL: https://www.frontendmentor.io/solutions/responsive-single-price-grid-component-using-css-grid-and-nextjs-hH_SBJj-io Live Site URL: https://single-price-grid-component-one-cyan.vercel.app
If you have any further questions or need assistance with implementing the hover feature, please feel free to reach out. Keep up the good work! š
Best regards, @AhsanCommits
0