AinaKhanDurrani
@AinaKhanDurraniAll comments
- @Aaliyan10Submitted 4 months ago@AinaKhanDurraniPosted 4 months ago
Your JavaScript handles form validation and submission effectively. For enhanced security, consider using textContent instead of innerHTML. Your code is readable and adheres to good practices however encapsulating reusable functions will improve maintainability.
0 - @Aaliyan10Submitted 4 months agoWhat specific areas of your project would you like help with?
I added a simple div banner in the share part. how do I add the shape that was mentioned in the design. And also how to change the color of the share button.
@AinaKhanDurraniPosted 4 months agoConsider using <article> for the main content to provide more semantic meaning, especially since it's an article preview component. Ensure all images have descriptive alt attributes, e.g., <img src="images/drawers.jpg" alt="Wooden drawers">.
For JavaScript, consider using more descriptive variable names and avoiding magic numbers (like 2 for toggling). For better readability, split the JavaScript code into functions. For example, create functions to handle showing and hiding the share options.
0