Skip to content

Commit bd6d99d

Browse files
Fix glitch in flags chart display height (#1880)
1 parent b60e9f7 commit bd6d99d

3 files changed

Lines changed: 4 additions & 4 deletions

File tree

lib/public/views/QcFlags/ForDataPass/QcFlagsForDataPassOverviewPage.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ export const QcFlagsForDataPassOverviewPage = ({
8989
}),
9090
mergeRemoteData([remoteQcFlags, remoteRun])
9191
.match({
92-
Success: ([flags, run]) => h('#qc-flags-chart-component', qcFlagsChartComponent(flags, run)),
92+
Success: ([flags, run]) => qcFlagsChartComponent(flags, run),
9393
Failure: () => null,
9494
Loading: () => spinner({ size: 2, absolute: false }),
9595
NotAsked: () => errorAlert([{ title: 'No data', detail: 'No QC flags or run data was asked for' }]),

lib/public/views/QcFlags/qcFlagsVisualization/qcFlagsChartComponent.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ import { h } from '/js/src/index.js';
2626
*/
2727
export const qcFlagsChartComponent = (flags, run) => {
2828
if ((run.qcTimeStart ?? null) === null || (run.qcTimeEnd ?? null) === null) {
29-
return h('h4.pv3', 'Missing run start or end, no QC flags chart');
29+
return h('h4#qc-flags-chart-component-error.pv3', 'Missing run start or end, no QC flags chart');
3030
}
3131

3232
/**

test/public/qcFlags/detailsForDataPass.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -168,11 +168,11 @@ module.exports = () => {
168168
it('should successfully display a message instead of QC flags chart when run is missing boundaries', async () => {
169169
await navigateToRunsPerDataPass(page, 1, 4, 5);
170170
await waitForNavigation(page, () => pressElement(page, '#row105-CPV-text a'));
171-
await expectInnerText(page, '#qc-flags-chart-component', 'Missing run start or end, no QC flags chart');
171+
await expectInnerText(page, '#qc-flags-chart-component-error', 'Missing run start or end, no QC flags chart');
172172

173173
await page.goBack();
174174

175175
await waitForNavigation(page, () => pressElement(page, '#row100-CPV-text a'));
176-
await expectInnerText(page, '#qc-flags-chart-component', 'Missing run start or end, no QC flags chart');
176+
await expectInnerText(page, '#qc-flags-chart-component-error', 'Missing run start or end, no QC flags chart');
177177
});
178178
};

0 commit comments

Comments
 (0)