split Tabs of Tabs and WithAnchor

Author: heymexa

bower.json

@@ -31,7 +31,8 @@
     "backbone-super": "~1.0.4",
     "backbone.view.elements": "~1.0.1",
     "backbone.factory": "~1.1.0",
-    "backbone.anchor": "~1.0.1"
+    "backbone.anchor": "~1.0.1",
+    "backbone.mix": "~1.0.1"
   },
   "devDependencies": {
     "mocha": "~2.0.1",

lib/Tabs.js

@@ -1,9 +1,8 @@
 define([
     'backbone',
     'Backbone.View.Elements',
-    'underscore',
-    'backbone.anchor/lib/anchor'
-], function (Backbone, ElementsView, _, anchor) {
+    'underscore'
+], function (Backbone, ElementsView, _) {
     'use strict';
 
     /**
@@ -73,35 +72,6 @@ define([
 
             this.getName = _.once(this.getName);
             this._initActiveTab();
-            this._linkWithAnchor();
-        },
-
-        /**
-         * @private
-         */
-        _linkWithAnchor: function () {
-            var name = this.getName();
-            anchor.on('change:' + name, this._onHashChange, this);
-            if (anchor.has(name)) {
-                this._processAnchorChange(anchor.get(name));
-            }
-        },
-
-        /**
-         * @param {string} tabName
-         * @protected
-         */
-        _processAnchorChange: function (tabName) {
-            this.show(tabName);
-        },
-
-        /**
-         * @param {Backbone.Model} model
-         * @param {string} tabName
-         * @private
-         */
-        _onHashChange: function (model, tabName) {
-            this._processAnchorChange(tabName);
         },
 
         /**
@@ -140,7 +110,6 @@ define([
                 return this;
             }
             this._setActiveTab(name);
-            anchor.set(this.getName(), name);
             return this;
         },
 

lib/TabsManager.js

@@ -1,8 +1,9 @@
 define([
     'underscore',
     'backbone.factory/lib/SelectorsFactory',
-    './Tabs'
-], function (_, SelectorsFactory, Tabs) {
+    './Tabs',
+    './WithAnchor'
+], function (_, SelectorsFactory, Tabs, WithAnchor) {
     'use strict';
 
     /**
@@ -17,7 +18,8 @@ define([
          */
         _selectors: function () {
             return _.defaults({
-                tabs: '.tabs'
+                tabs: '.tabs',
+                anchorTabs: '.tabs_with_anchor'
             }, this._super());
         },
 
@@ -28,7 +30,8 @@ define([
          */
         _products: function () {
             return _.defaults({
-                '*': Tabs
+                '*': Tabs,
+                anchorTabs: Tabs.mix(WithAnchor)
             }, this._super());
         },
 

lib/WithAnchor.js

@@ -0,0 +1,63 @@
+define([
+    'backbone.mix',
+    'backbone.anchor/lib/anchor',
+    'backbone-super'
+], function (Mixin, anchor) {
+    'use strict';
+
+    /**
+     * @mixin WithAnchors
+     * @extends Tabs
+     */
+    var WithAnchors = new Mixin(/** @lends WithAnchors# */{
+        /**
+         * @constructs
+         */
+        initialize: function () {
+            this._super();
+
+            this._linkWithAnchor();
+        },
+
+        /**
+         * @private
+         */
+        _linkWithAnchor: function () {
+            var name = this.getName();
+            anchor.on('change:' + name, this._onHashChange, this);
+            if (anchor.has(name)) {
+                this._processAnchorChange(anchor.get(name));
+            }
+        },
+
+        /**
+         * @param {string} tabName
+         * @protected
+         */
+        _processAnchorChange: function (tabName) {
+            this.show(tabName);
+        },
+
+        /**
+         * @param {Backbone.Model} model
+         * @param {string} tabName
+         * @private
+         */
+        _onHashChange: function (model, tabName) {
+            this._processAnchorChange(tabName);
+        },
+
+        /**
+         * @public
+         * @param {string} name
+         * @returns {Tabs} this
+         */
+        show: function (name) {
+            this._super(name);
+
+            anchor.set(this.getName(), name);
+            return this;
+        }
+    });
+    return WithAnchors;
+});