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.

Sign up for GitHub

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

Jump to bottom

Tags are case sensitive #165

Closed
nus-pe-bot opened this issue Nov 2, 2019 · 1 comment · Fixed by #187
Closed

Tags are case sensitive #165

nus-pe-bot opened this issue Nov 2, 2019 · 1 comment · Fixed by #187
Assignees
@gachia
Labels
severity.Low A flaw that is unlikely to affect normal operations of the product. Appears only very rarely. type.Enhancement An enhancement to an existing story.
Milestone
v1.4

Comments

@nus-pe-bot
Copy link

Not quite sure if this is a bug, but you may want to remove case sensitivity from the tagging

Screenshot 2019-11-01 at 4.38.41 PM.png

[original: tishyakhanna97/ped#5]
@ReignOfComputer ReignOfComputer added severity.Low A flaw that is unlikely to affect normal operations of the product. Appears only very rarely. type.Enhancement An enhancement to an existing story. labels Nov 2, 2019
@ReignOfComputer ReignOfComputer added this to the v1.4 milestone Nov 2, 2019
@gachia
Copy link

gachia commented Nov 2, 2019

This is a default feature from AB3. I can take over this to help lighten the load. @SQwQ @ReignOfComputer

@gachia gachia assigned gachia and unassigned SQwQ Nov 2, 2019
gachia added a commit to gachia/main that referenced this issue Nov 2, 2019
@gachia
fix: AY1920S1-CS2103T-F12-2#182, AY1920S1-CS2103T-F12-2#178, AY1920S1…
3435928 
…-CS2103T-F12-2#165, AY1920S1-CS2103T-F12-2#130

Fixed these issues.
@gachia gachia mentioned this issue Nov 2, 2019
Merged
gachia added a commit that referenced this issue Nov 2, 2019
@gachia
fix: #182, #178, #165, #130 (#187)
5cf55c6 
Fixed these issues.
fix: #182
fix: #178
fix: #165
fix: #130

Tags will always be in lower-case and will not have duplicates
Long input fixed in ResultDisplay. It will now wrap text.
Long input in CommandBox is now visible and not blocked by the Send button
All commands are now available in the help window. The user will still be required to see the User Guide for more detailed information.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gachia gachia

Labels
severity.Low A flaw that is unlikely to affect normal operations of the product. Appears only very rarely. type.Enhancement An enhancement to an existing story.
Projects
None yet
Milestone
v1.4
Development

Successfully merging a pull request may close this issue.

fix: #182, #178, #165, #130 gachia/main
4 participants
@ReignOfComputer @gachia @SQwQ @nus-pe-bot