@jhonmiccSubmitted over 2 years ago
Please give feedback on my result landing page.
Abdo Achhoubi
@abdoachhoubiAll comments
- @abdoachhoubiPosted over 2 years ago
Great work! Visually... But technically you still have some CSS errors that needs to be fixed, and also some accessibility issues...
0 - @PraiseImmanuelSubmitted over 2 years ago
- @jmontanholiSubmitted over 2 years ago
This is my solution for the challenge, please feel free to tell me how I can improve my solution.
- @BrianJ-27Submitted over 2 years ago
- @TobylarrySubmitted over 2 years ago@abdoachhoubiPosted over 2 years ago
Great work! But consider adding an alt attribute to your images even if you leave it empty... Because the accessibility issues have an important influence on the SEO.
0 - @Mobilnet-SolutionsSubmitted over 2 years ago@abdoachhoubiPosted over 2 years ago
Nice work! But I recommend you to fix the HTML issues and learn more about the Accessibility and ARIA in order to build apps that any human being can use! That's my feedback on the technical part! On the visual part, everything seems fine except the box shadow of the button. You can either learn more about it, or just use a box shadow generator to customize your own shadows! All the best and have fun coding ❤️
0 - @vianydevSubmitted over 2 years ago
- @Marvin-FigueroaSubmitted over 2 years ago@abdoachhoubiPosted over 2 years ago
Nice work! I have a suggestion if you don't mind. I guess you need to edit the function that handles the click of the accordion item so that it closes all the items and then opens the one that the user clicks on... This will prevent the opening of all the items which takes extra space that might ruin the design...
Marked as helpful0 - @pernorinSubmitted over 2 years ago
- @Juan13052001Submitted over 2 years ago@abdoachhoubiPosted over 2 years ago
Great work! You only need to fix some visual issues. Like the "per month" text color, and the button width...
Marked as helpful0 - @Jay035Submitted over 2 years ago@abdoachhoubiPosted over 2 years ago
Hi there! Great work, you only have to adjust the profile pictures!
0 - @Kwill3Submitted over 2 years ago
I couldn't figure out how to add the cyan line between the 3 boxes so if anyone could point me in the right direction I would be so grateful.
Please feel free to check (break) my code and give pointers. Cheers!
