Changeset 14329 for lang/javascript/userscripts/ldr_show_sbm_comments

Timestamp:

06/21/08 12:43:26 (5 months ago)

Author:

kga

Message:

fixed stupid mistake

Files:

• lang/javascript/userscripts/ldr_show_sbm_comments/ldr_show_sbm_comments.user.js (moved) (moved from lang/javascript/userscripts/ldr_show_sbm_comments/ldr_show_sbm_comments.js) (4 diffs)

lang/javascript/userscripts/ldr_show_sbm_comments/ldr_show_sbm_comments.user.js

@@ -3 +3 @@
 // @namespace      http://shinten.info/
 // @include        http://reader.livedoor.com/reader/
-// @resource       styles styles.css
+// @resource       styles http://svn.coderepos.org/share/lang/javascript/userscripts/ldr_show_sbm_comments/styles.css
 // @require        http://svn.coderepos.org/share/lang/javascript/jsdeferred/trunk/jsdeferred.userscript.js
 // ==/UserScript==
@@ -102 +102 @@
             updateStat();
             w.LoadEffect.Stop();
-            var c = [];
-            for (var s in o) c = c.concat(o[s]);
-            c = c.sort(function (a, b) {
+
+            var comments = merge(o).sort(function (a, b) {
                 return b.sort_key - a.sort_key;
-            });
+            }).map(function (c) { return c.comment }).join('\n');
+
             var d = w.$N('div', {
                 id    : 'sbm_' + item_id,
                 style : 'display: block;'
             });
-            d.innerHTML = c.map(function (cm) { return cm.comment }).join('\n');
+            d.innerHTML = comments;
+
             try { w.$('item_' + item_id).appendChild(d); } catch (e) {}
         }).error(function (e) {
@@ -120 +121 @@
     w.register_command('tc|togglecomment', function () {
         w.toggleClass('right_body', 'nocomment');
-        (w.contain(w.$('right_body').className, 'nocomment'))
-            ? w.message('Hide without comment')
-            : w.message('Show without comment');
+        w.message(
+            ((w.contain(w.$('right_body').className, 'nocomment')) ? 'Hide' : 'Show')
+            + ' without comment'
+        );
         setTimeout(function () {
             GM_setValue('hide', !eval(GM_getValue('hide')));
@@ -135 +137 @@
 }
 
+function merge (obj) {
+    var tmp = [];
+    for (var key in obj) tmp = tmp.concat(obj[key]);
+    return tmp;
+}
+
 })();