[273] Change alignment of numeric values in tables.
--- a/ckanext/ga_report/public/css/ga_report.css
+++ b/ckanext/ga_report/public/css/ga_report.css
@@ -45,3 +45,7 @@
width: 200px;
}
+.ga-reports-table .td-numeric {
+ text-align: center;
+}
+
--- a/ckanext/ga_report/public/scripts/ckanext_ga_reports.js
+++ b/ckanext/ga_report/public/scripts/ckanext_ga_reports.js
@@ -103,8 +103,8 @@
var handler = function(e) {
var target = $(e.delegateTarget);
var form = target.closest('form');
- form.attr('action', form.attr('action')+window.location.hash);
- form.submit();
+ var url = form.attr('action')+'?month='+target.val()+window.location.hash;
+ window.location = url;
};
var selectors = $('select[name="month"]');
assert(selectors.length>0);
--- a/ckanext/ga_report/templates/ga_report/ga_util.html
+++ b/ckanext/ga_report/templates/ga_report/ga_util.html
@@ -14,7 +14,7 @@
</select>
-<table py:def="social_table(items, with_source=False)" class="table table-condensed table-bordered table-striped">
+<table py:def="social_table(items, with_source=False)" class="ga-reports-table table table-condensed table-bordered table-striped">
<tr>
<th>Name</th>
<th py:if="with_source">Source</th>
@@ -24,7 +24,7 @@
<tr>
<td><a href="${url}">${name}</a></td>
<td py:if="with_source">${source}</td>
- <td>${count}</td>
+ <td class="td-numeric">${count}</td>
</tr>
</py:for>
</table>
@@ -45,15 +45,15 @@
</div>
-<table py:def="stat_table(items, title='Views')" class="table table-condensed table-bordered table-striped">
+<table py:def="stat_table(items, title='Views')" class="ga-reports-table table table-condensed table-bordered table-striped">
<tr>
<th>Name</th>
- <th>% ${title}</th>
+ <th class="td-numeric">% ${title}</th>
</tr>
<py:for each="name, value in items">
<tr>
<td>${name}</td>
- <td>${value}</td>
+ <td class="td-numeric">${value}</td>
</tr>
</py:for>
</table>
@@ -63,12 +63,13 @@
<div id="minornavigation-bg-left">
<div id="minornavigation-bg-right">
<ul class="nav nav-pills">
- <li py:attrs="{'class': 'active' if active_name=='Site-wide' else None}"><a py:attrs="{'class': 'active' if active_name=='Site-wide' else None}" href="${h.url_for(controller='ckanext.ga_report.controller:GaReport',action='index')}"><img src="/images/icons/page_white.png" height="16px" width="16px" alt="None" class="inline-icon "/> Site-wide</a></li>
+ <?python querystring='?month='+c.month if c.month else '' ?>
+ <li py:attrs="{'class': 'active' if active_name=='Site-wide' else None}"><a py:attrs="{'class': 'active' if active_name=='Site-wide' else None}" href="${h.url_for(controller='ckanext.ga_report.controller:GaReport',action='index')}${querystring}"><img src="/images/icons/page_white.png" height="16px" width="16px" alt="None" class="inline-icon "/> Site-wide</a></li>
<li py:attrs="{'class': 'active' if active_name=='Publishers' else None}">
- <a py:attrs="{'class': 'active' if active_name=='Publishers' else None}" href="${h.url_for(controller='ckanext.ga_report.controller:GaDatasetReport',action='publishers')}"><img src="/images/icons/page_white.png" height="16px" width="16px" alt="None" class="inline-icon "/> Publishers</a>
+ <a py:attrs="{'class': 'active' if active_name=='Publishers' else None}" href="${h.url_for(controller='ckanext.ga_report.controller:GaDatasetReport',action='publishers')}${querystring}"><img src="/images/icons/page_white.png" height="16px" width="16px" alt="None" class="inline-icon "/> Publishers</a>
</li>
<li py:attrs="{'class': 'active' if active_name=='Datasets' else None}">
- <a py:attrs="{'class': 'active' if active_name=='Datasets' else None}" href="${h.url_for(controller='ckanext.ga_report.controller:GaDatasetReport',action='read')}"><img src="/images/icons/page_white.png" height="16px" width="16px" alt="None" class="inline-icon "/> Datasets</a>
+ <a py:attrs="{'class': 'active' if active_name=='Datasets' else None}" href="${h.url_for(controller='ckanext.ga_report.controller:GaDatasetReport',action='read')}${querystring}"><img src="/images/icons/page_white.png" height="16px" width="16px" alt="None" class="inline-icon "/> Datasets</a>
</li>
</ul>
</div>
--- a/ckanext/ga_report/templates/ga_report/publisher/index.html
+++ b/ckanext/ga_report/templates/ga_report/publisher/index.html
@@ -27,6 +27,7 @@
<link rel="stylesheet" type="text/css" href="/css/ga_report.css"/>
<script type="text/javascript" src="/scripts/ckanext_ga_reports.js"></script>
<script type="text/javascript" src="/scripts/vendor/jquery.sparkline.modified.js"></script>
+ <script type="text/javascript" src="/scripts/rickshaw_ie7_shim.js"></script>
<script type="text/javascript" src="/scripts/vendor/d3.v2.js"></script>
<script type="text/javascript" src="/scripts/vendor/d3.layout.min.js"></script>
<script type="text/javascript" src="/scripts/vendor/rickshaw.min.js"></script>
@@ -47,17 +48,17 @@
</div>
</form>
- <table class="table table-condensed table-bordered table-striped">
+ <table class="ga-reports-table table table-condensed table-bordered table-striped">
<tr>
<th>Publisher</th>
- <th>Dataset Views</th>
+ <th class="td-numeric">Dataset Views</th>
</tr>
<py:for each="publisher, views, visits in c.top_publishers">
<tr>
<td>
${h.link_to(publisher.title, h.url_for(controller='ckanext.ga_report.controller:GaDatasetReport', action='read_publisher', id=publisher.name) + (("?month=" + c.month) if c.month else ''))}
</td>
- <td>${views}</td>
+ <td class="td-numeric">${views}</td>
</tr>
</py:for>
</table>
--- a/ckanext/ga_report/templates/ga_report/publisher/read.html
+++ b/ckanext/ga_report/templates/ga_report/publisher/read.html
@@ -12,6 +12,7 @@
<link rel="stylesheet" type="text/css" href="/css/ga_report.css"/>
<script type="text/javascript" src="/scripts/ckanext_ga_reports.js"></script>
<script type="text/javascript" src="/scripts/vendor/jquery.sparkline.modified.js"></script>
+ <script type="text/javascript" src="/scripts/rickshaw_ie7_shim.js"></script>
<script type="text/javascript" src="/scripts/vendor/d3.v2.js"></script>
<script type="text/javascript" src="/scripts/vendor/d3.layout.min.js"></script>
<script type="text/javascript" src="/scripts/vendor/rickshaw.min.js"></script>
@@ -61,7 +62,7 @@
</div>
</form>
<py:if test="c.top_packages">
- <table class="table table-condensed table-bordered table-striped">
+ <table class="ga-reports-table table table-condensed table-bordered table-striped">
<tr>
<th>Dataset</th>
<th>Views</th>
@@ -72,8 +73,8 @@
<td>
${h.link_to(package.title or package.name, h.url_for(controller='package', action='read', id=package.name))}
</td>
- <td>${views}</td>
- <td>${downloads}</td>
+ <td class="td-numeric">${views}</td>
+ <td class="td-numeric">${downloads}</td>
</tr>
</py:for>
</table>
--- a/ckanext/ga_report/templates/ga_report/site/index.html
+++ b/ckanext/ga_report/templates/ga_report/site/index.html
@@ -71,16 +71,16 @@
</ul>
<div class="tab-content">
<div class="tab-pane" id="totals">
- <table class="table table-condensed table-bordered table-striped">
+ <table class="ga-reports-table table table-condensed table-bordered table-striped">
<tr>
<th>Name</th>
- <th>Value</th>
+ <th class="td-numeric">Value</th>
<th>History</th>
</tr>
<py:for each="name, value, graph in c.global_totals">
<tr>
<td>${name}</td>
- <td>${value}</td>
+ <td class="td-numeric">${value}</td>
<td class="sparkline-cell">
<span class="sparkline" sparkTooltips="${','.join([x for x,y in graph])}">
${','.join([y for x,y in graph])}