please review thank you
MrByford
@MrByfordAll comments
- @Dinesh141197Submitted 12 months ago@MrByfordPosted 12 months ago
Styling and Fonts: Consider applying some additional styling and project-specific fonts to enhance the visual appeal and align the project closer to the intended design.
Background: Integrate the project's background color to create a cohesive and polished look, tying together the visual elements of the design.
Container Border: You might want to consider removing the border around the grid system container to achieve a cleaner and more seamless appearance.
Responsiveness: The responsiveness of the design is commendable! Explore adjusting the breakpoints based on the design spec sheet to ensure the design collapses at optimal points, aligning with the intended user experience.
Semantic HTML: Take a look into enhancing the HTML structure for better semantics. This could contribute to improved accessibility and SEO.
Overall Feedback: Fantastic effort on your project! There's room for improvement, as with any project, and you're on a great path. Keep up the good work, and continue refining your skills. Excited to see your progress!
Marked as helpful0 - @SamuelKRuan1901Submitted 12 months ago@MrByfordPosted 12 months ago
-
In reviewing the age calculator app, I observed that although the month value is correct, it displays an error message indicating that the field is required. However, the app still provides an output. It may be beneficial to revisit the logic behind this condition to prevent the error message from appearing when a correct entry is made.
-
Upon intentionally entering an invalid value, like 56 for days, the output appears as "NaN Days." Enhancing the user experience could involve addressing this issue to provide a more meaningful response or guiding the user towards a valid input.
-
Notably, when an invalid entry is submitted, there is no accompanying error message. Instead, the previous message disappears, potentially leading to the assumption that the entry is valid. For instance, entering the 56th day of a month contradicts the brief, which specifies not exceeding 31 days. Adding a clear error message would help users understand and correct their input.
0 -