Browse Source

updated setter syntax on title mixin

Rafał Pitoń 10 years ago
parent
commit
3ca288b73d
1 changed files with 8 additions and 10 deletions
  1. 8 10
      misago/emberapp/app/mixins/document-title.js

+ 8 - 10
misago/emberapp/app/mixins/document-title.js

@@ -1,15 +1,15 @@
+/* jshint unused:false */
 import Ember from 'ember';
 import Ember from 'ember';
 
 
 export default Ember.Mixin.create({
 export default Ember.Mixin.create({
-  title: function(key, value) {
-    // setter
-    if (arguments.length > 1) {
+  title: Ember.computed({
+    get(key) {
+      return document.title;
+    },
+    set(key, value) {
       this._changeTitle(value);
       this._changeTitle(value);
     }
     }
-
-    // getter
-    return document.title;
-  }.property(),
+  }),
 
 
   _changeTitle: function(title) {
   _changeTitle: function(title) {
     if (typeof title === 'string') {
     if (typeof title === 'string') {
@@ -26,8 +26,6 @@ export default Ember.Mixin.create({
       complete_title += ' | ' + title.parent;
       complete_title += ' | ' + title.parent;
     }
     }
 
 
-    complete_title += ' | ' + this.get('settings.forum_name');
-
-    document.title = complete_title;
+    document.title = complete_title + ' | ' + this.get('settings.forum_name');
   }
   }
 });
 });