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

Improve UI and aesthetics #121

Merged
merged 2 commits into from
Oct 24, 2021
Merged

Improve UI and aesthetics #121

merged 2 commits into from
Oct 24, 2021

Conversation

joshenx
Copy link

@joshenx joshenx commented Oct 23, 2021

UI: Tab Update!

New Features:

  • Appointments have two tabs, Upcoming and Archived

  • Application has a new Logo
    image

  • Text buttons have been replaced with icons

  • Selected cards have clearer markers

  • Appointment cards now split Patient and Appointment info

Current UI

image

@joshenx joshenx requested a review from didymental October 23, 2021 21:01
@joshenx joshenx added this to the v1.3 milestone Oct 23, 2021
@joshenx joshenx linked an issue Oct 23, 2021 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2021

Codecov Report

Merging #121 (dd9f7f2) into master (e5b3a2f) will decrease coverage by 0.23%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #121      +/-   ##
============================================
- Coverage     61.62%   61.38%   -0.24%     
  Complexity      491      491              
============================================
  Files            94       94              
  Lines          1793     1800       +7     
  Branches        190      190              
============================================
  Hits           1105     1105              
- Misses          629      636       +7     
  Partials         59       59              
Impacted Files Coverage Δ
...n/java/seedu/address/commons/core/GuiSettings.java 69.23% <ø> (ø)
src/main/java/seedu/address/ui/MainWindow.java 0.00% <0.00%> (ø)
src/main/java/seedu/address/ui/UiManager.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 e5b3a2f...dd9f7f2. Read the comment docs.

Copy link

@didymental didymental left a comment

Choose a reason for hiding this comment

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

LGTM

@didymental didymental merged commit d901dd5 into master Oct 24, 2021
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.

UI size seems too small upon initialisation
3 participants