Team Review - UI colors and fonts

Author: retiutut

OpenBCI_GUI/ConsoleLog.pde

@@ -65,7 +65,7 @@ static class ConsoleWindow extends PApplet implements Runnable {
 
         clipboardCopy = new ClipHelper();
         cp5 = new ControlP5(this);
-        PFont textAreaFont = createFont("Helvetica", 12, true);
+        PFont textAreaFont = createFont("Arial", 12, true);
         consoleTextArea = cp5.addTextarea("ConsoleWindow")
             .setPosition(0, headerHeight)
             .setSize(width, height - headerHeight)

OpenBCI_GUI/CustomCp5Classes.pde

@@ -216,9 +216,9 @@ public class MenuList extends controlP5.Controller {
                 menu.fill(127, 134, 143);
             }
             if (i == activeItem) {
-                menu.stroke(184, 220, 105, 255);
+                menu.stroke(TURN_ON_GREEN);
                 menu.strokeWeight(1);
-                menu.fill(184, 220, 105, 255);
+                menu.fill(TURN_ON_GREEN);
                 menu.rect(0, 0, getWidth()-1, itemHeight-1 );
                 menu.noStroke();
             } else {

OpenBCI_GUI/Debugging.pde

@@ -129,6 +129,7 @@ class HelpWidget {
     }
 
     private color getTextColor() {
+        /*
         switch (curOutputLevel) {
             case INFO:
                 return #00529B;
@@ -140,8 +141,10 @@ class HelpWidget {
                 return #D8000C;
             case DEFAULT:
             default:
-                return color(0, 5, 11);
+                return color(0, 5, 11); 
         }
+        */
+        return OPENBCI_DARKBLUE;
     }
 
     private color getBackgroundColor(int fadeVal) {  
@@ -155,6 +158,7 @@ class HelpWidget {
                 return color(199, sat, 97);
             case SUCCESS:
                 //base color -  #DFF2BF;
+                maxSat = 50;
                 sat = 25;
                 sat = (int)map(fadeVal, 0, 100, sat, maxSat);
                 return color(106, sat, 95);

OpenBCI_GUI/FilterUI.pde

@@ -547,7 +547,7 @@ class FilterUIPopup extends PApplet implements Runnable {
             .setPosition(_x, _y)
             .setCaptionLabel("")
             .setSize(_w, _h)
-            .setFont(createFont("Helvetica",12,true))
+            .setFont(createFont("Arial",12,true))
             .setFocus(false)
             .setColor(color(26, 26, 26))
             .setColorBackground(color(255, 255, 255)) // text field bg color

OpenBCI_GUI/OpenBCI_GUI.pde

@@ -238,7 +238,7 @@ final color buttonsLightBlue = color(57,128,204);
 final color GREY_125 = color(125);
 final color GREY_100 = color(100);
 final color GREY_20 = color(20);
-final color TURN_ON_GREEN = color(184,220,105);
+final color TURN_ON_GREEN = color(223, 242, 191);
 final color TURN_OFF_RED = color(224, 56, 45);
 final color BUTTON_HOVER = color(177, 184, 193);//color(252, 221, 198);
 final color BUTTON_HOVER_LIGHT = color(211, 222, 232);
@@ -337,6 +337,7 @@ void settings() {
     globalScreenResolution.append(" X ");
     globalScreenResolution.append(displayHeight);
     //Account for high-dpi displays on Mac, Windows, and Linux Machines Fixes #968
+    if (isMac())
     //pixelDensity(displayDensity());
     globalScreenDPI = new StringBuilder("High-DPI Screen Detected: ");
     globalScreenDPI.append(displayDensity() == 2);
@@ -349,12 +350,7 @@ void setup() {
 
     //V1 FONTS
     f1 = createFont("fonts/Raleway-SemiBold.otf", 16);
-    if (isMac()) {
-        f2 = createFont("Arial", 15, true);
-    } else {
-        f2 = createFont("fonts/Raleway-Regular.otf", 15);
-    }
-    
+    f2 = createFont("fonts/Raleway-Regular.otf", 15);
     f3 = createFont("fonts/Raleway-SemiBold.otf", 15);
     f4 = createFont("fonts/Raleway-SemiBold.otf", 64);  // clear bigger fonts for widgets
 

OpenBCI_GUI/SignalCheckThresholds.pde

@@ -45,7 +45,7 @@ class SignalCheckThresholdUI {
             .setPosition(_x, _y)
             .setCaptionLabel("")
             .setSize(_w, _h)
-            .setFont(createFont("Helvetica",12,true))
+            .setFont(p5)
             .setFocus(false)
             .setColor(color(26, 26, 26))
             .setColorBackground(color(255, 255, 255)) // text field bg color