-
Notifications
You must be signed in to change notification settings - Fork 6
error #106
Copy link
Copy link
Closed
Labels
area: ai-logicRelated to Gemini prompts, tokens, or model responses.Related to Gemini prompts, tokens, or model responses.area: frontendChanges specifically for the UI/Tailwind components.Changes specifically for the UI/Tailwind components.bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentationtypescriptType definition fixes, interfaces, or TS configuration.Type definition fixes, interfaces, or TS configuration.
Description
⚠️ Potential issue | 🟡 MinorAdd an accessible label for the language selector.
Line 75 renders a
<select>without a label. Screen readers will announce it generically, which hurts usability.♿ Proposed fix
- <select + <select value={language} onChange={(e) => setLanguage(e.target.value)} + aria-label="Select README language" className="bg-zinc-900/50 border border-white/10 rounded-2xl px-6 py-6 text-white focus:outline-none focus:ring-2 focus:ring-blue-500/50 transition-all backdrop-blur-xl appearance-none cursor-pointer min-w-[140px]" >📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.<select value={language} onChange={(e) => setLanguage(e.target.value)} aria-label="Select README language" className="bg-zinc-900/50 border border-white/10 rounded-2xl px-6 py-6 text-white focus:outline-none focus:ring-2 focus:ring-blue-500/50 transition-all backdrop-blur-xl appearance-none cursor-pointer min-w-[140px]" > {languages.map((lang) => ( <option key={lang} value={lang} className="bg-zinc-900 text-white"> {lang} </option> ))} </select>🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/components/Generator/SearchInput.tsx` around lines 75 - 85, The language <select> in the SearchInput component lacks an accessible label; add one by giving the <select> an id (e.g., "language-select") and either render a corresponding <label htmlFor="language-select">Language</label> (use your existing "sr-only" or visually-hidden utility class to keep it visually hidden) or add an aria-label/aria-labelledby on the <select>; ensure this change ties to the existing language, setLanguage and languages usage so screen readers announce the control properly.
Originally posted by @coderabbitai[bot] in #92
Reactions are currently unavailable
Metadata
Metadata
Labels
area: ai-logicRelated to Gemini prompts, tokens, or model responses.Related to Gemini prompts, tokens, or model responses.area: frontendChanges specifically for the UI/Tailwind components.Changes specifically for the UI/Tailwind components.bugSomething isn't workingSomething isn't workingdocumentationImprovements or additions to documentationImprovements or additions to documentationtypescriptType definition fixes, interfaces, or TS configuration.Type definition fixes, interfaces, or TS configuration.