change option name

mboinet · May 25, 2013 · ae06b97 · ae06b97
1 parent 8d5adc7
commit ae06b97
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 12 deletions.
diff --git a/src/index.html b/src/index.html
@@ -166,8 +166,8 @@ <h1>Settings</h1>
           id="articles-number" value="10" type="number">
         <input name="artOldestFirst" id="articles-oldest-first" type="checkbox">
         <label for="articles-oldest-first">Show oldest articles first</label>
-        <input name="hideEmptyCategories" id="hide-empty-categories" type="checkbox">
-        <label for="hide-empty-categories">Only show unread</label>
+        <input name="onlyUnread" id="only-unread" type="checkbox">
+        <label for="only-unread">Only show unread</label>
       </form>
     </div>
     <div class="version">

diff --git a/src/scripts/models.js b/src/scripts/models.js
@@ -41,7 +41,7 @@ define(['api','backbone','utils'],
     defaults: {
       articlesNumber: 10,
       articlesOldestFirst: false,
-      hideEmptyCategories: false
+      onlyUnread: false
     },
 
     validate: function(attrs, options){
@@ -89,7 +89,7 @@ define(['api','backbone','utils'],
         var request = {
           op:             "getCategories",
           enable_nested:  "false", // we want nested ones but they will not be nested yet
-          unread_only:    settings.attributes.hideEmptyCategories // get only feeds with unread articles
+          unread_only:    settings.attributes.onlyUnread // get only feeds with unread articles
         };
 
         api.ttRssApiCall(request, function(res){
@@ -132,7 +132,7 @@ define(['api','backbone','utils'],
           op:             "getFeeds",
           cat_id:         catId,
           include_nested: false,
-          unread_only: settings.attributes.hideEmptyCategories     // get only feeds with unread articles
+          unread_only: settings.attributes.onlyUnread     // get only feeds with unread articles
         };
 
         api.ttRssApiCall(
@@ -275,7 +275,7 @@ define(['api','backbone','utils'],
         var orderBy = settings.get("articlesOldestFirst") === true ? "date_reverse" : "feed_dates";
 
         // set view_mode depending on options
-        var viewMode = settings.get("hideEmptyCategories") ? "unread" : "adaptive";
+        var viewMode = settings.get("onlyUnread") ? "unread" : "adaptive";
 
         // we need to fetch the articles list for this feed
         var msg = {

diff --git a/src/scripts/views.js b/src/scripts/views.js
@@ -124,7 +124,7 @@ define(['jquery', 'models', 'templates','conf','utils'],
 
         var msg = "No categories";
 
-        if (models.settingsModel.get("hideEmptyCategories")){
+        if (models.settingsModel.get("onlyUnread")){
           msg = "No unread categories";
         }
 
@@ -397,7 +397,7 @@ define(['jquery', 'models', 'templates','conf','utils'],
 
         var msg = "No feeds";
 
-        if (models.settingsModel.get("hideEmptyCategories")){
+        if (models.settingsModel.get("onlyUnread")){
           msg = "No unread feeds";
         }
 
@@ -656,7 +656,7 @@ define(['jquery', 'models', 'templates','conf','utils'],
         // no elements in the collection
 
         // message depending on options
-        var msg = models.settingsModel.get("hideEmptyCategories") ?
+        var msg = models.settingsModel.get("onlyUnread") ?
           "No unread articles" :"No articles" ;
 
         this.$lv.html(tpl.roListElement({text: msg}));
@@ -1067,10 +1067,10 @@ define(['jquery', 'models', 'templates','conf','utils'],
     render: function(){
       var artNumber = this.model.get("articlesNumber");
       var artOldestFirst = this.model.get("articlesOldestFirst");
-      var hideEmptyCategories = this.model.get("hideEmptyCategories");
+      var onlyUnread = this.model.get("onlyUnread");
       this.$("#articles-number").attr("value", artNumber);
       this.$("#articles-oldest-first").prop("checked", artOldestFirst).checkboxradio("refresh");
-      this.$("#hide-empty-categories").prop("checked", hideEmptyCategories).checkboxradio("refresh");
+      this.$("#only-unread").prop("checked", onlyUnread).checkboxradio("refresh");
       return this;
     },
 
@@ -1081,7 +1081,7 @@ define(['jquery', 'models', 'templates','conf','utils'],
         {
           articlesNumber: $("#articles-number").val(),
           articlesOldestFirst: $("#articles-oldest-first").prop("checked"),
-          hideEmptyCategories: $("#hide-empty-categories").prop("checked")
+          onlyUnread: $("#only-unread").prop("checked")
         },
         {validate: true}
       );