Skip to content

Commit 7b1d885

Browse files
raifdmuellerclaude
andcommitted
fix: Fix lint warning and formatting issues
- Remove unused slogan1 variable (logo already shows brand name) - Fix Prettier formatting in onboarding-modal.js and main.js Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent fd1de79 commit 7b1d885

2 files changed

Lines changed: 17 additions & 9 deletions

File tree

website/src/components/onboarding-modal.js

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,8 @@ export function createOnboardingModal() {
1919

2020
const modal = document.createElement('div')
2121
modal.id = 'onboarding-modal'
22-
modal.className = 'fixed inset-0 bg-black/30 backdrop-blur-sm z-50 hidden items-center justify-center p-4'
22+
modal.className =
23+
'fixed inset-0 bg-black/30 backdrop-blur-sm z-50 hidden items-center justify-center p-4'
2324
modal.setAttribute('role', 'dialog')
2425
modal.setAttribute('aria-modal', 'true')
2526
modal.setAttribute('aria-labelledby', 'onboarding-title')
@@ -78,7 +79,6 @@ function buildModalContent() {
7879
// All text content is from trusted i18n translation files (not user input).
7980
// The YouTube video IDs are hardcoded constants above.
8081
const closeLabel = escapeHtml(i18n.t('modal.close'))
81-
const slogan1 = escapeHtml(i18n.t('onboarding.slogan1'))
8282
const slogan2 = escapeHtml(i18n.t('onboarding.slogan2'))
8383
const text1 = escapeHtml(i18n.t('onboarding.text1'))
8484
const text2 = escapeHtml(i18n.t('onboarding.text2'))
@@ -111,7 +111,9 @@ function buildModalContent() {
111111
<div class="px-6 pb-4">
112112
<div class="flex flex-col sm:flex-row gap-6">
113113
<div class="sm:w-1/2 flex-shrink-0 flex items-center justify-center">
114-
${window.matchMedia('(min-width: 640px)').matches ? `
114+
${
115+
window.matchMedia('(min-width: 640px)').matches
116+
? `
115117
<div class="rounded-xl overflow-hidden bg-black aspect-[9/16] max-h-[400px] w-full">
116118
<iframe
117119
src="${embedUrl}"
@@ -122,7 +124,8 @@ function buildModalContent() {
122124
allowfullscreen
123125
></iframe>
124126
</div>
125-
` : `
127+
`
128+
: `
126129
<a
127130
href="${youtubeUrl}"
128131
target="_blank"
@@ -134,7 +137,8 @@ function buildModalContent() {
134137
</svg>
135138
${watchVideo}
136139
</a>
137-
`}
140+
`
141+
}
138142
</div>
139143
140144
<div class="sm:w-1/2 flex flex-col gap-3 text-[var(--color-text)] text-sm leading-relaxed">

website/src/main.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -252,8 +252,10 @@ function bindRoleFilter() {
252252
if (other) other.value = e.target.value
253253
}
254254
})
255-
const searchQuery = document.getElementById('header-search-input')?.value
256-
|| document.getElementById('search-input')?.value || ''
255+
const searchQuery =
256+
document.getElementById('header-search-input')?.value ||
257+
document.getElementById('search-input')?.value ||
258+
''
257259
applyCardFilters(e.target.value, searchQuery)
258260
}
259261
})
@@ -278,8 +280,10 @@ function bindSearchInput() {
278280
if (query.trim()) {
279281
triggerSearchIndexBuild()
280282
}
281-
const roleId = document.getElementById('header-role-filter')?.value
282-
|| document.getElementById('role-filter')?.value || ''
283+
const roleId =
284+
document.getElementById('header-role-filter')?.value ||
285+
document.getElementById('role-filter')?.value ||
286+
''
283287
applyCardFilters(roleId, query)
284288
}
285289
})

0 commit comments

Comments
 (0)