Skip to content

Commit

Permalink
Merge branch 'maint-797893' into maint #789
Browse files Browse the repository at this point in the history
  • Loading branch information
christopherlam committed Sep 26, 2020
2 parents e84fd6a + d810e82 commit 6ecbb81
Show file tree
Hide file tree
Showing 5 changed files with 28 additions and 25 deletions.
23 changes: 16 additions & 7 deletions gnucash/report/html-chart.scm
Expand Up @@ -186,7 +186,6 @@
(cons 'display #t)
(cons 'labelString "")))
(cons 'ticks (list
(cons 'fontSize 12)
(cons 'maxRotation 30))))
;; the following another xAxis at the top
'((position . top)
Expand All @@ -205,7 +204,6 @@
(cons 'display 1.5)
(cons 'labelString "")))
(cons 'ticks (list
(cons 'fontSize 10)
(cons 'beginAtZero #f))))
;; the following another yAxis on the right
'((position . right)
Expand All @@ -215,7 +213,6 @@
))))
(cons 'title (list
(cons 'display #t)
(cons 'fontSize 16)
(cons 'fontStyle "")
(cons 'text ""))))))
"XXX" ;currency-iso
Expand Down Expand Up @@ -348,6 +345,17 @@ Chart.pluginService.register({
}
})

// copy font info from css into chartjs.
bodyStyle = window.getComputedStyle (document.querySelector ('body'));
Chart.defaults.global.defaultFontSize = parseInt (bodyStyle.fontSize);
Chart.defaults.global.defaultFontFamily = bodyStyle.fontFamily;
Chart.defaults.global.defaultFontStyle = bodyStyle.fontStyle;

titleStyle = window.getComputedStyle (document.querySelector ('h3'));
chartjsoptions.options.title.fontSize = parseInt (titleStyle.fontSize);
chartjsoptions.options.title.fontFamily = titleStyle.fontFamily;
chartjsoptions.options.title.fontStyle = titleStyle.fontStyle;

document.getElementById(chartid).onclick = function(evt) {
var activepoints = myChart.getElementAtEvent(evt);
var anchor = document.getElementById(jumpid);
Expand Down Expand Up @@ -398,21 +406,23 @@ document.getElementById(chartid).onclick = function(evt) {
(push (lambda (l) (set! retval (cons l retval))))
;; Use a unique chart-id for each chart. This prevents charts
;; clashing on multi-column reports
(id (guid-new-return)))
(id (symbol->string (gensym "chart"))))

(push (gnc:html-js-include
(gnc-path-find-localized-html-file "chartjs/Chart.bundle.min.js")))

;; the following hidden h3 is used to query style and copy onto chartjs
(push "<h3 style='display:none'></h3>")
(push (format #f "<div style='width:~a;height:~a;'>\n"
(size->str (gnc:html-chart-width chart))
(size->str (gnc:html-chart-height chart))))
(push (format #f "<a id='jump-~a' href='' style='display:none'></a>\n" id))
(push (format #f "<canvas id='chart-~a'></canvas>\n" id))
(push (format #f "<canvas id=~s></canvas>\n" id))
(push "</div>\n")
(push (format #f "<script id='script-~a'>\n" id))
(push (format #f "var curriso = ~s;\n" (gnc:html-chart-currency-iso chart)))
(push (format #f "var currsym = ~s;\n" (gnc:html-chart-currency-symbol chart)))
(push (format #f "var chartid = 'chart-~a';\n" id))
(push (format #f "var chartid = ~s;\n" id))
(push (format #f "var jumpid = 'jump-~a';\n" id))
(push (format #f "var loadstring = ~s;\n" (G_ "Load")))
(push (format #f "var chartjsoptions = ~a;\n\n"
Expand All @@ -428,7 +438,6 @@ document.getElementById(chartid).onclick = function(evt) {

(push "chartjsoptions.options.tooltips.callbacks.label = tooltipLabel;\n")
(push "chartjsoptions.options.tooltips.callbacks.title = tooltipTitle;\n")
(push "Chart.defaults.global.defaultFontFamily = \"'Trebuchet MS', Arial, Helvetica, sans-serif\";\n")
(push JS-setup)

(push "var myChart = new Chart(chartid, chartjsoptions);\n")
Expand Down
24 changes: 9 additions & 15 deletions gnucash/report/html-utilities.scm
Expand Up @@ -228,13 +228,10 @@


(define (gnc:html-make-generic-simple-warning report-title-string message)
(let ((p (gnc:make-html-text)))
(gnc:html-text-append!
p
(gnc:html-markup-h2 (string-append report-title-string ":"))
(gnc:html-markup-h2 "")
(gnc:html-markup-p message))
p))
(gnc:make-html-text
(gnc:html-markup-h3 (string-append report-title-string ":"))
(gnc:html-markup-h3 "")
(gnc:html-markup-p message)))


(define (gnc:html-make-options-link report-id)
Expand Down Expand Up @@ -301,14 +298,11 @@
(define (gnc:html-make-generic-warning
report-title-string report-id
warning-title-string warning-string)
(let ((p (gnc:make-html-text)))
(gnc:html-text-append!
p
(gnc:html-markup-h2 (string-append (G_ report-title-string) ":"))
(gnc:html-markup-h2 warning-title-string)
(gnc:html-markup-p warning-string)
(gnc:html-make-options-link report-id))
p))
(gnc:make-html-text
(gnc:html-markup-h3 (string-append (G_ report-title-string) ":"))
(gnc:html-markup-h3 warning-title-string)
(gnc:html-markup-p warning-string)
(gnc:html-make-options-link report-id)))

(define (gnc:html-make-generic-options-warning
report-title-string report-id)
Expand Down
2 changes: 1 addition & 1 deletion gnucash/report/reports/example/welcome-to-gnucash.scm
Expand Up @@ -37,7 +37,7 @@
(gnc:html-document-add-object!
doc
(gnc:make-html-text
(gnc:html-markup-h2
(gnc:html-markup-h3
(format #f (G_ "Welcome to GnuCash ~a !")
gnc:version))
(gnc:html-markup-p
Expand Down
2 changes: 1 addition & 1 deletion gnucash/report/reports/standard/new-owner-report.scm
Expand Up @@ -1079,7 +1079,7 @@ invoices and amounts.")))))
(gnc:html-document-add-object!
document
(gnc:make-html-text
(gnc:html-markup-h2 (G_ "No transactions found."))
(gnc:html-markup-h3 (G_ "No transactions found."))
(gnc:html-markup-p (G_ "No matching transactions found"))
(gnc:html-make-options-link (gnc:report-id report-obj)))))

Expand Down
2 changes: 1 addition & 1 deletion gnucash/report/reports/standard/price-scatter.scm
Expand Up @@ -192,7 +192,7 @@
(gnc:html-document-add-object!
document
(gnc:make-html-text
(gnc:html-markup-h2 title)
(gnc:html-markup-h3 title)
(gnc:html-markup-p text))))

(gnc:html-chart-set-type! chart 'line)
Expand Down

0 comments on commit 6ecbb81

Please sign in to comment.