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

Update User Guide for View, Close commands #121

Conversation

iamdiluxedbutcooler
Copy link

closes #105

@iamdiluxedbutcooler iamdiluxedbutcooler added this to the v1.4 milestone Oct 22, 2024
@iamdiluxedbutcooler iamdiluxedbutcooler self-assigned this Oct 22, 2024
@iamdiluxedbutcooler iamdiluxedbutcooler changed the title Update User Guide Update User Guide for View, Close commands Oct 22, 2024
Copy link

@itsme-zeix itsme-zeix left a comment

Choose a reason for hiding this comment

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

Great work! Thank you for updating the UG, especially with the formatting fixes/standardizations too!

Comment on lines +205 to +206
| `view` | Opens a split view showing detailed client information |
| `close` | Closes the split view of client details |

Choose a reason for hiding this comment

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

Perhaps we can standardize whether we want to use client details or detailed client information.

Choose a reason for hiding this comment

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

I would say detailed client information is more befitting of this context.

Copy link

@FionaQY FionaQY left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@ZShunRen ZShunRen left a comment

Choose a reason for hiding this comment

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

LGTM

@colinhia colinhia merged commit 5ce447d into AY2425S1-CS2103T-T14-4:master Oct 24, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update UG for View and Close command
5 participants