Skip to content

Commit 31076fb

Browse files
authored
Merge pull request #154 from gin0115/fix/gh152-fix-firefox-print-preview-artifacts__DEV
Added in css to ensure the clipping of bg image is hidden and doesnt …
2 parents 1aeeb59 + 51a307b commit 31076fb

File tree

6 files changed

+197
-93
lines changed

6 files changed

+197
-93
lines changed

themes/osi/assets/css/editor-style.css

Lines changed: 7 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

themes/osi/assets/css/editor-style.css.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 40 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
body > .wrapper {
1+
body>.wrapper {
22
background: transparent;
33
font-size: $textFontSize;
44
min-height: 100%;
@@ -8,9 +8,8 @@ body > .wrapper {
88
position: relative;
99
@include multi-transition('margin-left .2s ease-out, margin-right .2s ease-out, left .2s ease-out');
1010
z-index: 1;
11-
1211
&:after {
13-
content: '';
12+
content: '';
1413
background-color: $Nlightest;
1514
position: absolute;
1615
width: 100vw;
@@ -23,106 +22,95 @@ body > .wrapper {
2322
mask-repeat: no-repeat;
2423
z-index: -1;
2524
}
26-
2725
}
2826

27+
@media print {
28+
// Esnure the background is not printed
29+
body>.wrapper::after {
30+
content: none !important;
31+
display: none !important;
32+
visibility: hidden !important;
33+
}
34+
}
2935

3036
.content {
3137
padding: 0;
3238
// padding: 0 $smallPadding; // for a boxed width background with sidebar only
3339
padding-top: $paddingTopMobile; // for fixed-position header, adjust to match header size
34-
35-
@media only screen and ( max-width: 1200px ) {
40+
@media only screen and ( max-width: 1200px) {
3641
padding-top: 0;
3742
}
38-
3943
&.inner-padding {
4044
padding: 0 $smallPadding;
4145
}
42-
4346
.content-full .content--page {
4447
max-width: $boxedMax; // for a boxed width background with no sidebar
4548
padding: 0 $smallPadding; // for a boxed width background with no sidebar
4649
}
47-
4850
.content--page.no-title {
49-
padding-bottom: 0 !important;
50-
padding-top: 0 !important;
51+
padding-bottom: 0 !important;
52+
padding-top: 0 !important;
5153
}
52-
5354
.content--inner {
54-
max-width: $boxedMax; // for a boxed width background with no sidebar
55+
max-width: $boxedMax; // for a boxed width background with no sidebar
5556
}
56-
5757
&.has_sidebar {
58-
max-width: $boxedMax; // for a boxed width background with sidebar only
59-
padding: $paddingTopMobile $smallPadding 0 $smallPadding; // for a boxed width background with sidebar only
60-
61-
.content--inner {
62-
max-width: none;
63-
}
58+
max-width: $boxedMax; // for a boxed width background with sidebar only
59+
padding: $paddingTopMobile $smallPadding 0 $smallPadding; // for a boxed width background with sidebar only
60+
.content--inner {
61+
max-width: none;
62+
}
6463
}
6564
}
6665

6766
@media only screen and (min-width: #{$break-small}) {
68-
.content {
69-
&.inner-padding {
70-
padding: 0 $midPadding;
71-
}
72-
73-
.content-full .content--page {
74-
padding: 0 $midPadding; // for a boxed width background with no sidebar
75-
margin: 0 auto;
76-
}
77-
78-
&.has_sidebar {
79-
padding: $paddingTopMobile $midPadding 0 $midPadding; // for a boxed width background with sidebar only
80-
}
81-
}
67+
.content {
68+
&.inner-padding {
69+
padding: 0 $midPadding;
70+
}
71+
.content-full .content--page {
72+
padding: 0 $midPadding; // for a boxed width background with no sidebar
73+
margin: 0 auto;
74+
}
75+
&.has_sidebar {
76+
padding: $paddingTopMobile $midPadding 0 $midPadding; // for a boxed width background with sidebar only
77+
}
78+
}
8279
}
8380

8481
@media only screen and (min-width: #{$break-nav}) {
85-
86-
.content{
82+
.content {
8783
//padding-bottom:100px; // to account for position absolute footer
8884
// padding-top: $paddingTop; // for fixed-position header, adjust to match header size
89-
90-
.content-full .content--page{
85+
.content-full .content--page {
9186
padding: 0 $maxPadding; // for a boxed width background with no sidebar
9287
}
93-
9488
.content--page.no-title {
95-
padding-top: 0;
89+
padding-top: 0;
9690
}
97-
9891
&.has_sidebar {
99-
padding: $paddingTop $maxPadding 0 $maxPadding; // for a boxed width background with sidebar only
92+
padding: $paddingTop $maxPadding 0 $maxPadding; // for a boxed width background with sidebar only
10093
}
101-
10294
}
10395
}
10496

10597
@media only screen and (min-width: #{$break-xlarge}) {
106-
10798
.content {
10899
//padding-bottom: 150px; // to account for position absolute footer
109100
}
110-
111101
.content-main {
112102
padding-right: 1em;
113-
@include pinkgrid($colspan:9, $padside:3%);
114-
@include pinkrow($rowitems:2)
103+
@include pinkgrid($colspan: 9, $padside: 3%);
104+
@include pinkrow($rowitems: 2)
115105
}
116106
}
117107

118108
@media only screen and (min-width: #{$break-wide}) {
119109
.content {
120110
margin: 0 auto;
121-
// padding-top: $paddingTop; // for fixed-position header, adjust to match header size
111+
// padding-top: $paddingTop; // for fixed-position header, adjust to match header size
122112
}
123-
124113
.content--inner {
125-
margin: 0 auto;
114+
margin: 0 auto;
126115
}
127-
128-
}
116+
}

0 commit comments

Comments
 (0)