pjanthony2001 / pe

0 stars 0 forks source link

Minor cosmetics issues #12

Open pjanthony2001 opened 4 months ago

pjanthony2001 commented 4 months ago

There are slight issues with cosmetics in the UG as the UG was not optimized for pdf format. There are big chunks of whitespace in the history section.

image.png

image.png

nus-se-bot commented 4 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]

awkward formatting

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.


image.png the large blank space may hinder the reader a bit


[original: nus-cs2103-AY2324S2/pe-interim#5597] [original labels: type.DocumentationBug severity.VeryLow]

Their Response to the 'Original' Bug

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

This is a feature flaw in the conversion of github pages to PDF, and it is beyond our control.

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`] - [x] I disagree **Reason for disagreement:** Even though I agree with the developers that this issue might have been partly due to the Github Pages to PDF, more could have been done in the code to ensure that this white space does not occur, such as resizing the images to make it slightly smaller or placing the images side by side so that it would use up more horizontal width instead of height. It is clearly possible, and I do not think that "fixing it is less important than the work done in the current version of the product" as is required for it to be a Not In Scope. I do believe that this is a valid bug.