Conversation
mateacademy-ai-mentor
left a comment
There was a problem hiding this comment.
Review Summary
Great work on this task! Your solution correctly implements all the functional requirements, and the Sum component handles all prop variations perfectly. I am approving your submission.
I have one small suggestion for improvement in src/components/Sum/Sum.jsx. While your code works correctly, it's a best practice to use a single template literal for constructing strings with variables, as mentioned in the project checklist ([CHECKLIST ITEM #2]).
This improves readability. Here’s how you could write it:
<p>{`Sum of ${a} and ${b} is ${a + b}`}</p>This is just a stylistic point to keep in mind for future tasks. Overall, this is a solid implementation. Keep it up!
✨ Please rate this review in the Platform's notification center1. I will be grateful for your feedback! ✨
Footnotes
| export const Sum = ({ a = 0, b = 0 }) => ( | ||
| <div className="Card"> | ||
| <p> | ||
| Sum of {a} and {b} is {a + b} |
There was a problem hiding this comment.
This violates checklist item #2. While this works, the good practice recommended in the checklist is to use a single template literal (string interpolation) for the entire text content. For example: {Sum of ${a} and ${b} is ${a + b}}.
No description provided.