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 UI and UG #171

Merged
merged 3 commits into from
Nov 2, 2023
Merged

Conversation

wujy28
Copy link

@wujy28 wujy28 commented Nov 2, 2023

Summary of change

Updated Scrollbars of UI and Added sort and find to UG

Key changes

UG Updates

  1. Added sort and find
  2. Added department table
  3. Added new screenshots

Update UI

  1. Updated UI scrollbars
  2. Removed JavaFX warnings

Currently, the styling of the scrollbar looks out
of place with the rest of the UI. The help window's
font is different as well. Warnings are also thrown
when FXML files are loaded.

We should update the UI and remove these warnings
so that our GUI looks more cohesive and runs more
smoothly.

Let's,
* Style the scrollbar
* Style the help window
* Remove fxml warnings
Currently, the user guide is very outdated, lacking
commands we've added and the new UI screenshots.

We should update the UG to match our new iteration.

Let's,
* Update the Ui Screenshots
* Add the departments appendix
Currently, the user guide is outdated and does not contain
information about some newer commands.

We should make sure that our UG is up to date so that
users know what our app has to offer

Let's add the new sections into the UG.
@wujy28 wujy28 added the priority.High Must do label Nov 2, 2023
@wujy28 wujy28 added this to the v1.3b milestone Nov 2, 2023
@wujy28 wujy28 self-assigned this Nov 2, 2023
Copy link

@thienmy0 thienmy0 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for updating UI and UG!

Copy link

Choose a reason for hiding this comment

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

Thanks for updating the UG!

Copy link

Choose a reason for hiding this comment

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

Good job with the UI as usual!

@thienmy0 thienmy0 merged commit 7617c8a into AY2324S1-CS2103T-T14-2:master Nov 2, 2023
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants