kallithea Changeset - ecbb597c336c

Changeset - ecbb597c336c

Parent rev.

Child rev.

[Not reviewed]

default

0 2 0

Andrew Shadura - 11 years ago 2015-03-21 16:08:16
andrew@shadura.me

stats: fix display when no data ready yet

* jQ uses .append(), not .appendChild()
* when no data ready, display a nice message
* rephrase the message, the data isn't ready, not hasn't loaded yet

2 files changed with 6 insertions and 3 deletions:

kallithea/controllers/summary.py

kallithea/templates/summary/summary.html

0 comments (0 inline, 0 general)

kallithea/controllers/summary.py

➞

Show inline comments

@@ @@ -172,7 +172,7 @@ class SummaryController(BaseRepoControll @@
+            )
         else:
             c.no_data = True
-            c.trending_languages = json.dumps({})
+            c.trending_languages = json.dumps([])
         c.enable_downloads = c.db_repo.enable_downloads
         c.readme_data, c.readme_file = \
@@ @@ -196,7 +196,7 @@ class SummaryController(BaseRepoControll @@
     def statistics(self, repo_name):
         if c.db_repo.enable_statistics:
             c.show_stats = True
-            c.no_data_msg = _('No data loaded yet')
+            c.no_data_msg = _('No data ready yet')
         else:
             c.show_stats = False
             c.no_data_msg = _('Statistics are disabled for this repository')

kallithea/templates/summary/summary.html

➞

Show inline comments

@@ @@ -383,8 +383,11 @@ $(document).ready(function(){ @@
+        }
+    }
     if (data.length == 0) {
         tbl.innerHTML = "<tr><td>${_('No data ready yet')}</td></tr>";
+    }
-    $('#lang_stats').appendChild(tbl);
+    $('#lang_stats').append(tbl);
     $('#code_status_show_more').click(function(){
         $('.stats_hidden').show();
         $('#code_status_show_more').hide();

0 comments (0 inline, 0 general)