Skip to content

Commit b63a2d8

Browse files
authored
Merge pull request #438 from ford442/palette-add-aria-labels-modals-16587423975543865246
🎨 Palette: Add ARIA labels to buttons in Modals
2 parents da476c2 + 5513577 commit b63a2d8

2 files changed

Lines changed: 8 additions & 0 deletions

File tree

src/components/AISongModal.tsx

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -926,6 +926,7 @@ export function AISongModal({ isOpen, onClose, onImport, onShowToast, audioEngin
926926
<button
927927
onClick={handleFixCommonIssues}
928928
className="px-3 py-1 bg-red-900/50 hover:bg-red-800/50 text-red-300 text-xs rounded transition-all"
929+
aria-label="Fix Common JSON Issues"
929930
>
930931
🔧 Fix Issues
931932
</button>
@@ -935,6 +936,7 @@ export function AISongModal({ isOpen, onClose, onImport, onShowToast, audioEngin
935936
<button
936937
onClick={copyErrorReport}
937938
className="px-3 py-1 bg-gray-800 hover:bg-gray-700 text-gray-400 text-xs rounded transition-all"
939+
aria-label="Copy error details to clipboard"
938940
>
939941
{copiedError ? '✓ Copied!' : '📋 Copy'}
940942
</button>
@@ -1044,6 +1046,7 @@ export function AISongModal({ isOpen, onClose, onImport, onShowToast, audioEngin
10441046
<button
10451047
onClick={copyTemplate}
10461048
className="mb-3 px-4 py-2 bg-emerald-600 hover:bg-emerald-500 text-white text-xs font-medium rounded transition-all flex items-center gap-2"
1049+
aria-label="Copy AI Prompt Template"
10471050
>
10481051
<span>📋</span> Copy Template
10491052
</button>
@@ -1258,6 +1261,7 @@ export function AISongModal({ isOpen, onClose, onImport, onShowToast, audioEngin
12581261
className="px-4 py-2 bg-emerald-600 hover:bg-emerald-500 text-white text-sm font-medium rounded transition-all flex items-center gap-2 disabled:opacity-50 disabled:cursor-not-allowed"
12591262
onClick={() => onShowToast('Audio preview coming soon!', 'info')}
12601263
disabled
1264+
aria-label="Play Preview (Coming Soon)"
12611265
>
12621266
<span></span> Play Preview
12631267
</button>
@@ -1345,6 +1349,7 @@ export function AISongModal({ isOpen, onClose, onImport, onShowToast, audioEngin
13451349
onClick={handleClose}
13461350
disabled={isImporting}
13471351
className="px-3 sm:px-4 py-2 bg-gray-800 hover:bg-gray-700 text-gray-300 text-xs font-medium rounded transition-all disabled:opacity-50"
1352+
aria-label="Cancel Import"
13481353
>
13491354
<span className="hidden sm:inline">Cancel</span>
13501355
<span className="sm:hidden"></span>
@@ -1359,6 +1364,7 @@ export function AISongModal({ isOpen, onClose, onImport, onShowToast, audioEngin
13591364
? 'bg-emerald-600 hover:bg-emerald-500 text-white shadow-[0_0_20px_rgba(16,185,129,0.3)] hover:shadow-[0_0_30px_rgba(16,185,129,0.5)]'
13601365
: 'bg-gray-700 text-gray-500'
13611366
}`}
1367+
aria-label="Import AI Song"
13621368
>
13631369
{isImporting ? (
13641370
<>

src/components/RbsImportModal.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -448,6 +448,7 @@ export function RbsImportModal({ isOpen, onClose, onImport, onShowToast }: RbsIm
448448
onClick={() => fileInputRef.current?.click()}
449449
className="mt-2 px-4 py-2 bg-gray-800 hover:bg-gray-700 text-gray-300 text-xs rounded transition-all"
450450
disabled={isParsing}
451+
aria-label="Browse Files to Import"
451452
>
452453
Browse Files
453454
</button>
@@ -838,6 +839,7 @@ export function RbsImportModal({ isOpen, onClose, onImport, onShowToast }: RbsIm
838839
onClick={onClose}
839840
disabled={isImporting}
840841
className="px-4 py-2 bg-gray-800 hover:bg-gray-700 text-gray-300 text-xs font-medium rounded transition-all disabled:opacity-50"
842+
aria-label="Cancel Import"
841843
>
842844
Cancel
843845
</button>

0 commit comments

Comments
 (0)