diff options
author | Mark Wells <mark@freeside.biz> | 2016-07-12 13:51:47 -0700 |
---|---|---|
committer | Mark Wells <mark@freeside.biz> | 2016-07-12 13:51:47 -0700 |
commit | 64aa9aa36d8b8636b4984de7d15cd7bd1a889542 (patch) | |
tree | 721f6e4ed783b8279d802a64f7b078329aaca4bb | |
parent | e215fc213a1eb9a3365cecf5ce82f1a58f154cd6 (diff) |
workaround for page-break-inside:avoid on Chrome, #71301
-rw-r--r-- | httemplate/elements/freeside.css | 5 | ||||
-rw-r--r-- | httemplate/elements/printtofit.js | 17 | ||||
-rw-r--r-- | httemplate/elements/table-grid.html | 2 | ||||
-rw-r--r-- | httemplate/search/elements/grouped-search/html | 2 |
4 files changed, 23 insertions, 3 deletions
diff --git a/httemplate/elements/freeside.css b/httemplate/elements/freeside.css index cc104a196..87dec3dd4 100644 --- a/httemplate/elements/freeside.css +++ b/httemplate/elements/freeside.css @@ -342,4 +342,7 @@ div.package-marker-change_from { display: inline-block; } - +/* but see printtofit.js */ +.nosplit, .nosplitrows tr { + page-break-inside: avoid; +} diff --git a/httemplate/elements/printtofit.js b/httemplate/elements/printtofit.js index 66257fca8..98236026e 100644 --- a/httemplate/elements/printtofit.js +++ b/httemplate/elements/printtofit.js @@ -1,13 +1,30 @@ $().ready(function() { + // yuck + var isChrome = /Chrome/.test(navigator.userAgent) && /Google Inc/.test(navigator.vendor); var beforePrint = function() { if ($('body').width() > 0) { // 7.5 inches * 96 DPI; maybe make the width a user pref? var maxwidth = 7.5 * 96; $('body').css('zoom', maxwidth / $('body').width()); + if (isChrome) { + // Chrome doesn't respect page-break-* styles on table rows/cells, + // so wrap the contents of table cells with a block element + // ref. Chromium bug #99124, #87828, #59193 + // should be fixed with Chrome 53 + var nosplits = $('.nosplitrows td'); + if (nosplits.length > 0) { + nosplits.wrapInner('<div class="nosplit autowrap" />'); + } + } } }; var afterPrint = function() { $('body').css('zoom', 1); + // get the direct children of the wrapper divs. + var nosplits = $('div.autowrap >'); + if (nosplits.length > 0) { + nosplits.unwrap(); + } } if (window.matchMedia) { // chrome, most importantly; also IE10? diff --git a/httemplate/elements/table-grid.html b/httemplate/elements/table-grid.html index 4f4200570..bf60fe03e 100644 --- a/httemplate/elements/table-grid.html +++ b/httemplate/elements/table-grid.html @@ -1,4 +1,4 @@ -<TABLE CLASS="grid" CELLSPACING=<% $opt{cellspacing} %> CELLPADDING=<% $opt{cellpadding} %> <% $opt{bgcolor} %>> +<TABLE CLASS="grid nosplitrows" CELLSPACING=<% $opt{cellspacing} %> CELLPADDING=<% $opt{cellpadding} %> <% $opt{bgcolor} %>> <%init> diff --git a/httemplate/search/elements/grouped-search/html b/httemplate/search/elements/grouped-search/html index df1471a52..2ecb68815 100644 --- a/httemplate/search/elements/grouped-search/html +++ b/httemplate/search/elements/grouped-search/html @@ -120,7 +120,7 @@ for (my $i = 0; $i < $group_info->{num}; $i++) { border-spacing: 0; } </STYLE> -<table class="grid"> +<table class="grid nosplitrows"> <thead> <& /search/elements/search-html.html:header_row, 'header' => $opt{'header'}, |