From 16fc8d3ffd9d3d14748f5adc085397bfcd7efda9 Mon Sep 17 00:00:00 2001
From: Jordan Eldredge <jordaneldredge@gmail.com>
Date: Sun, 16 Nov 2014 17:59:26 -0800
Subject: [PATCH] Convert Font to object literal

---
 font.js   | 25 ++++++++++++-------------
 winamp.js |  2 +-
 2 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/font.js b/font.js
index 08e2f79c..b52e2c40 100644
--- a/font.js
+++ b/font.js
@@ -1,20 +1,20 @@
 // Manage rendering text from this skin's text.bmp file
-Font = function() {
+Font = {
 
     // Fill a node with a <div> containing character <div>s
-    this.setNodeToString = function(node, string) {
+    setNodeToString: function(node, string) {
         stringElement = this._stringNode(string);
         node.innerHTML = '';
         node.appendChild(stringElement);
-    }
+    },
 
     // Get a <div> containing char
-    this.characterNode = function(char) {
+    characterNode: function(char) {
         return this.displayCharacterInNode(char, document.createElement('div'));
-    }
+    },
 
     // Style/populate a <div> to display a character
-    this.displayCharacterInNode = function(character, node) {
+    displayCharacterInNode: function(character, node) {
         position = this._getCharPosition(character);
         row = position[0];
         column = position[1];
@@ -28,30 +28,29 @@ Font = function() {
 
         node.innerHTML = character;
         return node;
-    }
+    },
 
     // Get a <div> containing character <div>s
-    this._stringNode = function(string) {
+    _stringNode: function(string) {
         parentDiv = document.createElement('div');
         for (var i = 0, len = string.length; i < len; i++) {
             char = string[i].toLowerCase();
             parentDiv.appendChild(this.characterNode(char));
         }
         return parentDiv;
-    }
-
+    },
 
     // Find the background offsets for a given character
-    this._getCharPosition = function(char) {
+    _getCharPosition: function(char) {
         position = this._fontLookup[char];
         if(!position) {
             return this._fontLookup[' '];
         }
         return position;
-    }
+    },
 
     /* TODO: There are too many " " and "_" characters */
-    this._fontLookup = {
+    _fontLookup: {
         "a": [0,0], "b": [0,1], "c": [0,2], "d": [0,3], "e": [0,4], "f": [0,5],
         "g": [0,6], "h": [0,7], "i": [0,8], "j": [0,9], "k": [0,10],
         "l": [0,11], "m": [0,12], "n": [0,13], "o": [0,14], "p": [0,15],
diff --git a/winamp.js b/winamp.js
index 821230e2..00957255 100755
--- a/winamp.js
+++ b/winamp.js
@@ -4,7 +4,7 @@ function Winamp () {
     this.fileManager = new FileManager();
     this.media = new Media('player');
     this.skinManager = SkinManager;
-    this.font = new Font();
+    this.font = Font;
 
     this.nodes = {
         'option': document.getElementById('option'),