Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent overlap of task title and checkbox, set min length of task panel #197

Merged

Conversation

mingyi456
Copy link

No description provided.

* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp: (31 commits)
  Fix PR issues
  Change ... to …
  Fix grammar
  Fix grammar
  Improve command parsing and error handling
  Fix AY2122S1-CS2103-F09-2#134, AY2122S1-CS2103-F09-2#135, AY2122S1-CS2103-F09-2#136, AY2122S1-CS2103-F09-2#143
  Fix AY2122S1-CS2103-F09-2#158
  Fix AY2122S1-CS2103-F09-2#151
  Fix AY2122S1-CS2103-F09-2#125, AY2122S1-CS2103-F09-2#139
  Fix AY2122S1-CS2103-F09-2#128
  Fix AY2122S1-CS2103-F09-2#112, AY2122S1-CS2103-F09-2#110
  Fix clear undo
  Update Model javadoc
  Fix KeywordTaskFilter hasConflictWith method
  Change execute and undo methods
  Fix ListTaskCommand
  Deconflict task filters
  Add test cases for `commons.util.StringUtil.limitString()`
  Bug fix: DoneTaskCommand undo/redo
  Remove unused import in DoneTaskFilter
  ...
* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp: (29 commits)
  Fix test cases
  Add isDone to EditTaskDescriptor
  Fix set task command
  feat: Add starting tasks on init
  Fix checkstyle again
  Add various tests, redo test
  Add SetTaskCommand undo test
  Fix checkstyle
  Add tests
  Add EOF to Fonts.css
  Absorb enter in EditTaskDialog
  Update help window
  Remove duplicate line in ModelManager
  Remove trailing whitespace
  Fix task filter bugs
  Update model docs
  Fix GUI task editing not being undoable
  Refactor Command to UndoableCommand
  Update default button css
  Remove command input border
  ...
@mingyi456 mingyi456 requested a review from jeffsieu November 5, 2021 12:09
@mingyi456 mingyi456 added this to the v1.4 milestone Nov 5, 2021
@mingyi456 mingyi456 linked an issue Nov 5, 2021 that may be closed by this pull request
@mingyi456 mingyi456 marked this pull request as ready for review November 5, 2021 12:55
@codecov-commenter
Copy link

codecov-commenter commented Nov 5, 2021

Codecov Report

Merging #197 (ccf8b03) into master (dfaf4cd) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #197      +/-   ##
============================================
- Coverage     61.74%   61.70%   -0.05%     
  Complexity      693      693              
============================================
  Files           123      123              
  Lines          2766     2768       +2     
  Branches        266      266              
============================================
  Hits           1708     1708              
- Misses          964      966       +2     
  Partials         94       94              
Impacted Files Coverage Δ
src/main/java/seedu/address/ui/TaskCard.java 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dfaf4cd...ccf8b03. Read the comment docs.

yeppog
yeppog previously requested changes Nov 6, 2021
Copy link

@yeppog yeppog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2021-11-06.21-08-38.mp4

Odd slider behavior when dragging about the middle resize bar. TaskCard expands when the TaskList is expanded, but it does not shrink when the TaskList is shrunk.

Above issue needs to be fixed.

* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp: (28 commits)
  Fix trailing whitespace at EOF
  Fix trailing whitespaces
  Fix stuff
  Add PPP
  Fix checkstyle
  Skeleton for PPP
  Changes the UG
  Remove whitespaces
  Add contacts DG
  Update UI png and minor edits to AboutUs and index
  Update error message
  Add Contacts in UCP use cases
  Grammar fix
  Fix broken links
  Update EditTaskCommand Docs
  Pull from upstream
  Update use cases
  docs: Update PPP for yeppog
  Add monospace to PR link
  Update PPP Jeff Sieu
  ...
# Conflicts:
#	src/main/java/seedu/address/ui/TaskCard.java
@mingyi456 mingyi456 requested a review from yeppog November 6, 2021 14:43
@mingyi456 mingyi456 dismissed yeppog’s stale review November 7, 2021 09:50

Updated the PR

Copy link

@yeppog yeppog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@yeppog yeppog merged commit 61865a2 into AY2122S1-CS2103-F09-2:master Nov 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PE-D] Long task names gets blocked by check box
3 participants