Skip to content

Commit d1276c3

Browse files
committed
change display when missing flag
1 parent 9cd525c commit d1276c3

1 file changed

Lines changed: 7 additions & 9 deletions

File tree

lib/public/views/QcFlags/ActiveColumns/gaqFlagsActiveColumns.js

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -32,19 +32,17 @@ const formatGeneralQuality = (contributingFlags, gaqDetectors) => {
3232
const allGood = flagTypes.every(({ bad }) => !bad);
3333
const someBadNotReproducible = flagTypes.some(({ bad, mcReproducible }) => bad && !mcReproducible);
3434

35-
const flagForSomeDetectorIsMissing = gaqDetectors.length !== contributingFlags.length;
3635
const missingVerifications = contributingFlags.some(({ verifications }) => verifications.length === 0);
37-
const missingDataMessage = [
38-
... flagForSomeDetectorIsMissing ? [h('', 'No flag for some detector')] : [],
39-
... missingVerifications ? [h('', 'At least one flag is not verified')] : [],
40-
];
41-
42-
const missingDataTooltip = missingDataMessage.length
43-
? h('.d-inline-block.m1', tooltip(iconWarning(), h('.flex-column', missingDataMessage)))
36+
const missingDataTooltip = missingVerifications
37+
? h('.d-inline-block.m1', tooltip(iconWarning(), h('.flex-column', h('', 'At least one flag is not verified'))))
4438
: null;
4539

40+
const flagForSomeDetectorIsMissing = gaqDetectors.length !== contributingFlags.length;
41+
4642
let qualityDisplay = null;
47-
if (allGood) {
43+
if (flagForSomeDetectorIsMissing) {
44+
qualityDisplay = h('.gray-darker.m2.badge', tooltip(iconBan(), 'No flag for some detector'));
45+
} else if (allGood) {
4846
qualityDisplay = badge('good', { color: QcSummaryColors.ALL_GOOD });
4947
} else if (someBadNotReproducible) {
5048
qualityDisplay = badge('bad', { color: QcSummaryColors.ALL_BAD });

0 commit comments

Comments
 (0)