adjust ui: colors, jingle, icon, fix whitespace #98
Conversation
sigmund-c
added
type.Enhancement
Codecov Report
@@ Coverage Diff @@
## master #98 +/- ##
============================================
- Coverage 59.26% 59.02% -0.24%
Complexity 580 580
============================================
Files 119 120 +1
Lines 2251 2260 +9
Branches 264 264
============================================
Hits 1334 1334
- Misses 846 855 +9
Partials 71 71
Continue to review full report at Codecov.
|
sigmund-c
removed
the
severity.Medium
src/main/resources/view/McTheme2.css
Outdated
| @@ -0,0 +1,393 @@ | |||
| .background { | |||
There was a problem hiding this comment.
Is this file the alternative color scheme? Maybe can make use of UserPrefs to allow users to switch to this. Also would be good if the duplicates in the two themes are condensed into one place so that we don't need to edit both when including new GUI elements. Otherwise there will be two representations of some parts of our GUI styling.
| #shiftAssignments .label, | ||
| #workerAssignments .label, | ||
| #unavailableTiming .label, | ||
| #leaveShiftAssignments .label, | ||
| #leaveWorkerAssignments .label { |
There was a problem hiding this comment.
This needs to be updated with current master branch code.
| @@ -0,0 +1,24 @@ | |||
| package seedu.address.ui; | |||
There was a problem hiding this comment.
Not sure if its because of my system but the jingle seems to get cut off halfway through when playing on startup.
…into ui-adjustments
…into ui-adjustments # Conflicts: # src/main/java/mcscheduler/ui/JinglePlayer.java
*ps. This PR is gonna be a bit annoying, sorry
Fixes #61
Added McDonald's Jingle (annoying part) #67
Changed GUI to white/yellow/red (also annoying) #69
Changed app icon to McD's #71