Week 4 - Day 4: Peer to Peer code review ๐๐
Closed this issue ยท 0 comments
ahmedeid6842 commented
Hi @Gh-Reza ๐๐
Great job on your work so far! While you have done an excellent job with the design, there are still some issues that need to be addressed. Keep up the good work! ๐ช๐ช
To highlight ๐ฏ
- Good commit history ๐
- No linter errors.โ๏ธ
- Nice design. ๐
Changes That Need to be Made โป๏ธ
- Enhance the clarity and engagement of the Live Demo section in the README file.
- Changes to be made in the contact section :
- In the
<textarea>
element, there are twoname
attributes. You should remove the extra name attribute so that the element only has one. - For accessibility reasons, it is recommended to use
<label>
elements for each of the form inputs. This helps screen readers to identify the purpose of each form input.
- To improve the user experience, consider making the background non-scrollable when the popup page is open. You can achieve this by applying the CSS property
overflow: hidden;
. - Consider addressing the issue of the work card cover image appearing pixelated.