YuvBindal / pe

0 stars 0 forks source link

Text Overflow for address on edit #8

Open YuvBindal opened 4 months ago

YuvBindal commented 4 months ago

The UI allows overflow of the text when a long address is typed out in the edit command. As shown in the picture below,

Screenshot 2024-04-19 at 5.02.01 PM.png

Screenshot 2024-04-19 at 5.02.07 PM.png

Perhaps, the team can think of adding a scroll element to the UI to rectify this issue similar to the previous issues.

nus-se-script commented 4 months ago

[IMPORTANT!: Please do not edit or reply to this comment using the GitHub UI. You can respond to it using CATcher during the next phase of the PE]

Team's Response

No details provided by team.

The 'Original' Bug

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

Field name is pushed into oblivion by field variable

Description

Screenshot 2024-04-19 at 16.58.02.png

Oversized fields push the field name itself into oblivion. Phone, address and email names are gone.

Fix

Implement the field name within a fixed UI block so that it will not be pushed away by other parts of the component.


[original: nus-cs2103-AY2324S2/pe-interim#2298] [original labels: type.FunctionalityBug severity.Low]

Their Response to the 'Original' Bug

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

As mentioned in our planned enhancements, we plan to improve the UI to accommodate excessively long fields. In fact, the screenshot we included here shows an example of excessively long address like the example you have provided :) image.png

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 reason]


## :question: Issue response Team chose [`response.Rejected`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]
## :question: Issue type Team chose [`type.FunctionalityBug`] Originally [`type.FeatureFlaw`] - [ ] I disagree **Reason for disagreement:** [replace this with your reason]