From 944267b7947f64a6f22442a1c714def7f32253c1 Mon Sep 17 00:00:00 2001 From: William Allen Date: Wed, 16 Sep 2026 10:56:46 -0400 Subject: [PATCH] Add "macOS" to list of known operating system names The build summary card currently only shows an apple icon for the operating system names "Darwin" and "OSX". CTest reports the name "macOS" for the latest version of macOS. This updates the list to include "macOS" in the list of apple names, and refactors the relevant code to be more concise. --- .../components/shared/BuildSummaryCard.vue | 30 +++++++++++-------- 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/resources/js/vue/components/shared/BuildSummaryCard.vue b/resources/js/vue/components/shared/BuildSummaryCard.vue index 018d7c51de..2da542382a 100644 --- a/resources/js/vue/components/shared/BuildSummaryCard.vue +++ b/resources/js/vue/components/shared/BuildSummaryCard.vue @@ -29,17 +29,8 @@ class="tw-truncate tw-min-w-0" > - - - {{ build.operatingSystemName }} {{ build.operatingSystemRelease }} @@ -181,10 +172,25 @@ export default { }, methods: { - fullHumanReadableDateTimeString(timestamp) { return DateTime.fromISO(timestamp).toLocaleString(DateTime.DATETIME_FULL_WITH_SECONDS); }, + + iconFromOperatingSystemName(operatingSystemName) { + switch (operatingSystemName) { + case 'Windows': + return this.FA.faWindows; + // TODO: Add more specific Linux types. May require CTest work. + case 'Linux': + return this.FA.faLinux; + case 'Darwin': + case 'OSX': + case 'macOS': + return this.FA.faApple; + default: + return undefined; + } + }, }, };