search

Cazh1 / pe

0 stars 0 forks source link

Novice User Target #7

Open Cazh1 opened 10 months ago

Cazh1 commented 10 months ago

The application cleverly handles the problem of both sides having the same pieces by using capitol and non-capitol letters. However, since this targeted towards new players, they might get confused easily especially when pieces such as P and p may look similar.

image.png

Since the application is clearly able to show color, it may serve new players better to be able to easily differentiate between opponent and own pieces, especially at a glance. It will also help them to easily notice and identify opponent pieces, especially like when a opponent pawn is about to reach the end for promotion.

soc-pe-bot commented 10 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Hard to distinguish between player and cpu pawn

The pawn at c6 belongs to me, but however since p and P are similar, it is quite hard to distinguish between the player pawn and the cpu pawn.

image.png

[original: nus-cs2113-AY2324S1/pe-interim#439] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

This is valid that capitalized and non-capitalized characters are hard to distinguish, however, the representation of the chess pieces is already explicitly stated in the UG and the legend command.

Also, the CLI nature of the application makes it challenging to display the pieces in other formats. Hence, this is the best representation we can implement while ensuring that the color of the pieces are distinguishable.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue response Team chose [`response.NotInScope`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.VeryLow`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]