Any feedback on the code will be super appreciated.
Rachana Hegde
@rachanahegdeAll comments
- @roguepantherSubmitted over 3 years ago@rachanahegdePosted over 3 years ago
This looks pretty good - only issue is that your HTML code isn't indented properly in a few places but that can be addressed easily. I think that could be the reason for the accessibility issue mentioned in the report.
1 - @fotinh0Submitted over 3 years ago
Any suggestions or critiques are highly appreciated!
@rachanahegdePosted over 3 years agoThis looks really good and it's clearly responsive! Also nice work organising your code, it's readable and easy to navigate. Overall, great job.
Marked as helpful1 - @playerzer0-uiSubmitted over 3 years ago
wait why it looked cracked in the screenshot but nice in the live site?
@rachanahegdePosted over 3 years agoHi! The screenshot looks fine. Nice work! To fix your accessibility issues, you need to add some semantic HTML tags like <header> and <main>.
1 - @thantko20Submitted over 3 years ago
Hi. I think I messed up at centering the flex container vertically. Is there any way to center the container vertically?
@rachanahegdePosted over 3 years agoHi! This looks good but to make your buttons look more like the buttons in the design, you can modify the font-size and make it 15px. This is the same font size as the paragraphs.
Marked as helpful1