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
Changed color label FINALLY #68
Conversation
ITS WORKING!
Added screenshots and updated the DG
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.
Some minor changes for next round.
@@ -86,6 +86,8 @@ private void loadDivePage(DiveSession dive) { | |||
dateTime.setText(String.format(FORMAT_TIME_NOW, dive.getDateStart().getOurDateString())); | |||
checkPressureGrp(dive.getPressureGroupAtBeginning().getPressureGroup()); | |||
currentDive = dive; | |||
pgStart.getStyleClass().add(checkPressureGrp(dive.getPressureGroupAtBeginning().getPressureGroup())); | |||
pgEnd.getStyleClass().add(checkPressureGrp(dive.getPressureGroupAtEnd().getPressureGroup())); |
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.
you need to remove existing style classes too!!
@@ -124,64 +126,65 @@ private String dateTimeSend() { | |||
* Receives the pressure group that needs to be parsed | |||
* sets the attribute on the screen depending on the Pressure group. | |||
*/ | |||
private void checkPressureGrp(String pgGrp) { | |||
private String checkPressureGrp(String pgGrp) { | |||
switch(pgGrp) { | |||
case "A": |
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.
You might want to check in the standard if it is possible to make this shorter.
case "B": | ||
break; | ||
return "thisIsGreen"; |
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.
Declare these as constants!!
ITS WORKING!