search

AY2223S1-CS2103T-W08-4 / tp

AB3 for Team Project
https://ay2223s1-cs2103t-w08-4.github.io/tp/
MIT License
1 stars 5 forks source link

Fix bug where elipsis is shown due to long text #250

Closed Kang-Quan closed 2 years ago

Kang-Quan commented 2 years ago

Fix bug for other potential field to have long text to enable word wrap

codecov-commenter commented 2 years ago

Codecov Report

Merging #250 (df4f8f6) into master (cddfb24) will decrease coverage by 0.10%. The diff coverage is 20.00%.

@@             Coverage Diff              @@
##             master     #250      +/-   ##
============================================
- Coverage     68.67%   68.56%   -0.11%     
- Complexity      709      711       +2     
============================================
  Files           112      112              
  Lines          2493     2510      +17     
  Branches        304      306       +2     
============================================
+ Hits           1712     1721       +9     
- Misses          669      677       +8     
  Partials        112      112
Impacted Files Coverage Δ
src/main/java/seedu/address/ui/PersonCard.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/SupplyItemCard.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/TaskCard.java 0.00% <0.00%> (ø)
...u/address/logic/parser/EditStockCommandParser.java 91.30% <100.00%> (ø)
...a/seedu/address/logic/commands/AddItemCommand.java 88.23% <0.00%> (ø)
...edu/address/logic/parser/AddItemCommandParser.java 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Kang-Quan commented 2 years ago

Problems caused by very long inputs are considered functionality bugs. image

Fixing this issue should not violate the feature freeze. image