Changeset 4bab6c1 in OpenWorkouts-current

Timestamp:

Jan 29, 2019, 12:02:12 PM (5 years ago)

Author:

borja <borja@…>

Branches:

current, feature/docs, master

Children:

bed4f06

Parents:

7783f97

Message:

Use the chart_selector variable when looking for the element where the week chart

will be rendered (instead of the hardcoded "svg")

Location:

ow

Files:

• static/js/ow.js (modified) (2 diffs)
• templates/dashboard.pt (modified) (1 diff)

ow/static/js/ow.js

@@ -31 +31 @@
     var openstreetmap_attr = 'Map data &copy; <a href="http://www.osm.org">OpenStreetMap</a>';
 
-    // Some constants reused through the code
+    // Some vars reused through the code
     var map;
     var gpx;
@@ -147 +147 @@
            Build a d3 bar chart, populated with data from the given url.
          */
-        var chart = d3.select("svg"),
+        var chart = d3.select(chart_selector),
             margin = {top: 20, right: 20, bottom: 30, left: 50},
             width = +chart.attr("width") - margin.left - margin.right,

ow/templates/dashboard.pt

@@ -211 +211 @@
     <script type="text/javascript">
      var week_chart = owjs.week_chart({
-         chart_selector: 'div.js-week-stats',
+         chart_selector: 'div.js-week-stats svg',
          url: "${request.resource_url(context, 'week')}",
          current_day_name: "${current_day_name}"