-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update DG with Command History and Command Assistant #221
Update DG with Command History and Command Assistant #221
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM except for minor changes. You not adding diagrams?
docs/DeveloperGuide.md
Outdated
|
||
The user keystroke will be read in `CommandBox#handleKeyStroke`. The user input will be evaluated by `SystemCommand#execute` to determine the appropriate help message to display. | ||
|
||
Given below is an example usage scenario and how the Find mechanism behaves at each step. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be command assistant feature
docs/DeveloperGuide.md
Outdated
|
||
The user keystroke will be read in `CommandBox#handleKeyStroke`, which will call the appropriate method in `CommandHistory` to retrieve the previous and next commands. The retrieved commands are passed into `CommandBox#commandTextField` to be displayed. | ||
|
||
Given below is an example usage scenario and how the Find mechanism behaves at each step. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be command history feature
docs/DeveloperGuide.md
Outdated
|
||
Step 2. The user enters `find ` to search for a matching contact. | ||
|
||
Step 3. `CommandBox#handleKeyStroke` reads `find ` keystroke and calls `SystemCommand#execute`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is find
a keystroke or a String?
Perhaps the space is a keystroke but it reads the String
Codecov Report
@@ Coverage Diff @@
## master #221 +/- ##
=========================================
Coverage 70.75% 70.75%
Complexity 672 672
=========================================
Files 97 97
Lines 2086 2086
Branches 246 246
=========================================
Hits 1476 1476
Misses 519 519
Partials 91 91 Continue to review full report at Codecov.
|
No description provided.