search

kaijie0102 / ceg-cs2113-pe

0 stars 0 forks source link

Financial Statement no longer usable when i have more than 100 entries #9

Open kaijie0102 opened 11 months ago

kaijie0102 commented 11 months ago

image.png

nus-pe-script commented 11 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]

Table is not well formatted when the console window is small

Steps to reproduce:

  1. Add some income records.
  2. Run the viewIncomes command.

Expected:

  • The table fits the size of the console window. This can be done by removing extra spaces.

Actual:

  • The table is too large and some characters is pushed to the next line.

Screenshots:

image.png

image.png

[original: nus-cs2113-AY2324S1/pe-interim#889] [original labels: type.FunctionalityBug severity.VeryLow]

Their Response to the 'Original' Bug

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

This issue would not happen if the application is run on full screen. Dynamic sizing is used to maintain the UI and better presentation for users.

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.Rejected`] - [x] I disagree **Reason for disagreement:** The small app has only a couple of features and viewing financial statement is one of them. In the case where UI is messed up, the app is rendered almost useless.
## :question: Issue type Team chose [`type.DocumentationBug`] Originally [`type.FeatureFlaw`] - [x] I disagree **Reason for disagreement:** It is not a simple matter of documentation, where by the project is "explained". The team did not account for the case where inputs are longer than expected and thus it is a feature flaw.
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Medium`] - [x] I disagree **Reason for disagreement:** The small app has only a couple of features and viewing financial statement is one of them. In the case where UI is messed up, the app is rendered almost useless.