corrections in SidebarToggler and HistoryTabs

src/javascripts/modules/HistoryTabs.js

@@ -27,7 +27,7 @@ PurpleMine.HistoryTabs = (function()
         this.$history       = $("#history");
         this.lang           = document.documentElement.lang;
 
-        if ("undefined" === translations[this.lang])
+        if (typeof translations[this.lang] === "undefined")
         {
             this.lang = "en";
         }

src/javascripts/modules/SidebarToggler.js

@@ -5,6 +5,14 @@ PurpleMine.SidebarToggler = (function()
     "use strict";
 
     var self; // Make it work for browsers without Function.prototype.bind
+    var translations = {
+        en: {
+            toggler: "Toggle sidebar"
+        },
+        pl: {
+            toggler: "Pokaż/ukryj panel boczny"
+        }
+    };
 
     function SidebarToggler()
     {
@@ -15,6 +23,14 @@ PurpleMine.SidebarToggler = (function()
         this.$toggler       = null;
         this.$main          = $("#main");
         this.$sidebar       = $("#sidebar");
+        this.lang           = document.documentElement.lang;
+
+        if (typeof translations[this.lang] === "undefined")
+        {
+            this.lang = "en";
+        }
+
+        this._ = translations[this.lang];
 
         // Fix issue with context menu position
         if ("relative" === this.$main.css("position"))
@@ -58,14 +74,11 @@ PurpleMine.SidebarToggler = (function()
 
     SidebarToggler.prototype.buildButton = function()
     {
-        var togglerLabel = document.documentElement.lang === "pl" ?
+        var togglerClass = "sidebar-toggler",
-                            "Pokaż/ukryj panel boczny" :
-                            "Toggle sidebar",
-            togglerClass = "sidebar-toggler",
             togglerHtml;
 
         togglerHtml = "<a href=\"javascript:;\" class=\"" + togglerClass +
-                        "\" title=\"" + togglerLabel + "\"></a>";
+                        "\" title=\"" + this._.toggler + "\"></a>";
         this.$toggler = $(togglerHtml);
 
         this.$main.append(this.$toggler);

src/sass/javascripts/_sidebar-toggler.scss

@@ -1,8 +1,6 @@
 @if $enable-sidebar-toggler {
     $toggler-position-v:        1px;
     $toggler-position-h:        1px;
-    $arrow-right:               $fa-var-caret-square-o-right;
-    $arrow-left:                $fa-var-caret-square-o-left;
 
     #main {
         position: relative;
@@ -72,7 +70,7 @@
 
             @if $use-font-awesome {
                 &:before {
-                    content: $arrow-left;
+                    content: $fa-var-caret-square-o-left;
                 }
             } @else {
                 @extend %image-arrow-left;
@@ -82,7 +80,7 @@
 
             @if $use-font-awesome {
                 &:before {
-                    content: $arrow-right;
+                    content: $fa-var-caret-square-o-right;
                 }
             } @else {
                 @extend %image-arrow-right;
@@ -93,7 +91,7 @@
             @if $sidebar-position == "left" {
                 @if $use-font-awesome {
                     &:before {
-                        content: $arrow-right;
+                        content: $fa-var-caret-square-o-right;
                     }
                 } @else {
                     @extend %image-arrow-right;
@@ -101,7 +99,7 @@
             } @else {
                 @if $use-font-awesome {
                     &:before {
-                        content: $arrow-left;
+                        content: $fa-var-caret-square-o-left;
                     }
                 } @else {
                     @extend %image-arrow-left;