Skip to content

Commit 5297da5

Browse files
Add expander to log output
1 parent abde929 commit 5297da5

File tree

4 files changed

+88
-22
lines changed

4 files changed

+88
-22
lines changed

Diff for: src/layout/css/style.scss

+44-14
Original file line numberDiff line numberDiff line change
@@ -123,22 +123,51 @@ span.xpassed,
123123
$extra-height: 240px;
124124
$extra-media-width: 320px;
125125

126-
.log {
127-
background-color: #e6e6e6;
128-
border: $border-width solid #e6e6e6;
129-
color: black;
130-
display: block;
131-
font-family: 'Courier New', Courier, monospace;
132-
max-height: $extra-height - 2 * $spacing;
133-
overflow-y: scroll;
134-
padding: $spacing;
135-
white-space: pre-wrap;
136-
137-
&:only-child {
138-
height: inherit;
126+
.logwrapper {
127+
max-height: $extra-height - 2 * $spacing;
128+
overflow-y: scroll;
129+
background-color: #e6e6e6;
130+
&.expanded {
131+
max-height: none;
132+
.logexpander {
133+
&:after {
134+
content: 'collapse [-]';
135+
}
136+
}
137+
}
138+
.logexpander {
139+
z-index: 1;
140+
position: sticky;
141+
top: 10px;
142+
width: max-content;
143+
border: 1px solid;
144+
border-radius: 3px;
145+
padding: 5px 7px;
146+
margin: 10px 0 10px calc(100% - 80px);
147+
cursor: pointer;
148+
background-color: #e6e6e6;
149+
&:after {
150+
content: 'expand [+]';
151+
}
152+
&:hover {
153+
color: #000;
154+
border-color: #000;
155+
}
156+
}
157+
.log {
158+
min-height: 40px;
159+
position: relative;
160+
top: -50px;
161+
height: calc(100% + 50px);
162+
border: $border-width solid #e6e6e6;
163+
color: black;
164+
display: block;
165+
font-family: 'Courier New', Courier, monospace;
166+
padding: $spacing;
167+
padding-right: 80px;
168+
white-space: pre-wrap;
139169
}
140170
}
141-
142171
div.media {
143172
border: $border-width solid #e6e6e6;
144173
float: right;
@@ -200,6 +229,7 @@ div.media {
200229
}
201230

202231
.col-result {
232+
width: 130px;
203233
&:hover::after {
204234
content: ' (hide details)';
205235
}

Diff for: src/pytest_html/resources/index.jinja2

+4-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,10 @@
5353
<div class="media__name"></div>
5454
<div class="media__counter"></div>
5555
</div>
56-
<div class="log"></div>
56+
<div class="logwrapper">
57+
<div class="logexpander"></div>
58+
<div class="log"></div>
59+
</div>
5760
</td>
5861
</tr>
5962
</tbody>

Diff for: src/pytest_html/resources/style.css

+38-6
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: src/pytest_html/scripts/main.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
const { dom, findAll } = require('./dom.js')
1+
const { dom, find, findAll } = require('./dom.js')
22
const { manager } = require('./datamanager.js')
33
const { doSort } = require('./sort.js')
44
const { doFilter } = require('./filter.js')
@@ -67,6 +67,7 @@ const renderContent = (tests) => {
6767
rows.forEach((row) => {
6868
if (!!row) {
6969
findAll('.collapsible td:not(.col-links', row).forEach(addItemToggleListener)
70+
find('.logexpander', row).addEventListener('click', (evt) => evt.target.parentNode.classList.toggle('expanded'))
7071
newTable.appendChild(row)
7172
}
7273
})

0 commit comments

Comments
 (0)