review footprint iconography and messages

This commit is contained in:
Christoph Kluge 2024-12-04 16:09:06 +01:00
parent 257250714d
commit 01c06728eb
2 changed files with 23 additions and 15 deletions

View File

@ -109,21 +109,21 @@
return { return {
...fmBase, ...fmBase,
color: "danger", color: "danger",
message: `Metric average way ${fmc.lowerIsBetter ? "above" : "below"} expected normal thresholds.`, message: `Footprint value way ${fmc.lowerIsBetter ? "above" : "below"} expected normal threshold.`,
impact: 3 impact: 3
}; };
} else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "caution")) { } else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "caution")) {
return { return {
...fmBase, ...fmBase,
color: "warning", color: "warning",
message: `Metric average ${fmc.lowerIsBetter ? "above" : "below"} expected normal thresholds.`, message: `Footprint value ${fmc.lowerIsBetter ? "above" : "below"} expected normal threshold.`,
impact: 2, impact: 2,
}; };
} else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "normal")) { } else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "normal")) {
return { return {
...fmBase, ...fmBase,
color: "success", color: "success",
message: "Metric average within expected thresholds.", message: "Footprint value within expected thresholds.",
impact: 1, impact: 1,
}; };
} else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "peak")) { } else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "peak")) {
@ -131,7 +131,7 @@
...fmBase, ...fmBase,
color: "info", color: "info",
message: message:
"Metric average above expected normal thresholds: Check for artifacts recommended.", "Footprint value above expected normal threshold: Check for artifacts recommended.",
impact: 0, impact: 0,
}; };
} else { } else {
@ -139,7 +139,7 @@
...fmBase, ...fmBase,
color: "secondary", color: "secondary",
message: message:
"Metric average above expected peak threshold: Check for artifacts!", "Footprint value above expected peak threshold: Check for artifacts!",
impact: -1, impact: -1,
}; };
} }
@ -201,10 +201,14 @@
> >
<div class="mx-1"> <div class="mx-1">
<!-- Alerts Only --> <!-- Alerts Only -->
{#if fpd.impact === 3 || fpd.impact === -1} {#if fpd.impact === 3}
<Icon name="exclamation-triangle-fill" class="text-danger" /> <Icon name="exclamation-triangle-fill" class="text-danger" />
{:else if fpd.impact === 2} {:else if fpd.impact === 2}
<Icon name="exclamation-triangle" class="text-warning" /> <Icon name="exclamation-triangle" class="text-warning" />
{:else if fpd.impact === 0}
<Icon name="info-circle" class="text-info" />
{:else if fpd.impact === -1}
<Icon name="info-circle-fill" class="text-danger" />
{/if} {/if}
<!-- Emoji for all states--> <!-- Emoji for all states-->
{#if fpd.impact === 3} {#if fpd.impact === 3}
@ -214,7 +218,7 @@
{:else if fpd.impact === 1} {:else if fpd.impact === 1}
<Icon name="emoji-smile" class="text-success" /> <Icon name="emoji-smile" class="text-success" />
{:else if fpd.impact === 0} {:else if fpd.impact === 0}
<Icon name="emoji-laughing" class="text-info" /> <Icon name="emoji-smile" class="text-info" />
{:else if fpd.impact === -1} {:else if fpd.impact === -1}
<Icon name="emoji-dizzy" class="text-danger" /> <Icon name="emoji-dizzy" class="text-danger" />
{/if} {/if}

View File

@ -114,21 +114,21 @@
return { return {
...fmBase, ...fmBase,
color: "danger", color: "danger",
message: `Metric average way ${fmc.lowerIsBetter ? "above" : "below"} expected normal thresholds.`, message: `Footprint value way ${fmc.lowerIsBetter ? "above" : "below"} expected normal threshold.`,
impact: 3 impact: 3
}; };
} else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "caution")) { } else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "caution")) {
return { return {
...fmBase, ...fmBase,
color: "warning", color: "warning",
message: `Metric average ${fmc.lowerIsBetter ? "above" : "below"} expected normal thresholds.`, message: `Footprint value ${fmc.lowerIsBetter ? "above" : "below"} expected normal threshold.`,
impact: 2, impact: 2,
}; };
} else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "normal")) { } else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "normal")) {
return { return {
...fmBase, ...fmBase,
color: "success", color: "success",
message: "Metric average within expected thresholds.", message: "Footprint value within expected thresholds.",
impact: 1, impact: 1,
}; };
} else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "peak")) { } else if (evalFootprint(jf.value, fmt, fmc.lowerIsBetter, "peak")) {
@ -136,7 +136,7 @@
...fmBase, ...fmBase,
color: "info", color: "info",
message: message:
"Metric average above expected normal thresholds: Check for artifacts recommended.", "Footprint value above expected normal threshold: Check for artifacts recommended.",
impact: 0, impact: 0,
}; };
} else { } else {
@ -144,7 +144,7 @@
...fmBase, ...fmBase,
color: "secondary", color: "secondary",
message: message:
"Metric average above expected peak threshold: Check for artifacts!", "Footprint value above expected peak threshold: Check for artifacts!",
impact: -1, impact: -1,
}; };
} }
@ -264,10 +264,14 @@
id={`footprint-${job.jobId}-${index}`} id={`footprint-${job.jobId}-${index}`}
> >
<div class="mx-1"> <div class="mx-1">
{#if fpd.impact === 3 || fpd.impact === -1} {#if fpd.impact === 3}
<Icon name="exclamation-triangle-fill" class="text-danger" /> <Icon name="exclamation-triangle-fill" class="text-danger" />
{:else if fpd.impact === 2} {:else if fpd.impact === 2}
<Icon name="exclamation-triangle" class="text-warning" /> <Icon name="exclamation-triangle" class="text-warning" />
{:else if fpd.impact === 0}
<Icon name="info-circle" class="text-info" />
{:else if fpd.impact === -1}
<Icon name="info-circle-fill" class="text-danger" />
{/if} {/if}
{#if fpd.impact === 3} {#if fpd.impact === 3}
<Icon name="emoji-frown" class="text-danger" /> <Icon name="emoji-frown" class="text-danger" />
@ -276,7 +280,7 @@
{:else if fpd.impact === 1} {:else if fpd.impact === 1}
<Icon name="emoji-smile" class="text-success" /> <Icon name="emoji-smile" class="text-success" />
{:else if fpd.impact === 0} {:else if fpd.impact === 0}
<Icon name="emoji-laughing" class="text-info" /> <Icon name="emoji-smile" class="text-info" />
{:else if fpd.impact === -1} {:else if fpd.impact === -1}
<Icon name="emoji-dizzy" class="text-danger" /> <Icon name="emoji-dizzy" class="text-danger" />
{/if} {/if}