BBButton: subtle variant with neutral color has an opaque white background, so it renders white on white in dark mode
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- react, typescript
- Ambito
- frontend
Direzione di ricerca
Inizia leggendo le definizioni delle varianti di Button mostrate in dist/components/Button.js e individua il codice sorgente e i test corrispondenti alla variante neutra subtle di BBButton. Confrontala con subtle.default e verifica il comportamento esistente delle variabili del tema. Il lavoro è completato quando il pulsante neutro subtle ha uno sfondo trasparente, testo leggibile e lo sfondo grigio chiaro specificato al passaggio del mouse, senza modificare altre varianti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
The subtle + neutral combination uses the same colours as primary.neutral: an opaque --color-neutral-white background with --color-gray text. Every other subtle button has a transparent background, as in subtle.default.
In the BigBlueButton HTML5 client's dark theme, --color-gray turns white. --color-neutral-white is deliberately left alone, because the library also uses it as the label colour over brand backgrounds. The button ends up white on white.
Found on the pre-flight "Settings" button (bigbluebutton/bigbluebutton#25842).
Steps to reproduce
- Render
<BBButton variant="subtle" color="neutral" label="Settings" onClick={...} />on a dark surface. - Override
--color-graywith a light colour, as the client's dark theme does.
Expected: a transparent background, as on the other subtle buttons, with readable text.
Actual: a white pill with near-invisible text. In the light theme the white pill also stands out on any surface that isn't white.
Cause
From dist/components/Button.js (0.8.0):
subtle: {
default: { color: R, iconColor: N, background: "transparent", border: !1, hoverColor: R, hoverBackground: E, hoverBorder: !1, outline: N },
neutral: { color: C, background: k, border: !1, hoverColor: C, hoverBackground: S, hoverBorder: !1, outline: B }, // same as primary.neutral
}
// k = --color-neutral-white, C = --color-gray, S = --color-brand-aux, E = --color-light-gray
Suggested fix
Make subtle.neutral transparent like subtle.default and change only the text colour:
neutral: { color: C, iconColor: C, background: "transparent", border: !1, hoverColor: C, hoverBackground: E, hoverBorder: !1, outline: N },
The hover then uses --color-light-gray, which themes can already override.
Workaround
The client uses variant="subtle" without color.
- Lingua principale
- TypeScript
- Stelle
- 2
- Fork
- 5
- Merge medio
- 2g 5h
- PR unite (30g)
- 9
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di bigbluebutton/bbb-ui-components-react
-
Add support for dark themeApertaenhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
-
Add standard component for notifications (BBBToast)Forse di nuovo libera @Arthurk12 l’ha presa 31 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
bigbluebutton/bbb-ui-components-react#72 · 1 assegnatario ·
-
enhancement
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
Tutte le issue di bigbluebutton/bbb-ui-components-react
Issue simili
-
bot:ai-assisted status:untriaged
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
midnightntwrk/midnight-js#1424 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
mksglu/context-mode#1268 ·
I maintainer di solito rispondono entro 5 giorni
-
Edit:Apertacheck:failed streams:edit
Difficoltà 2/5 1-3 ore Idoneità per principianti 60/100
iptv-org/iptv#54352 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
[Table] reserveSelectedRowOnPaginate=false 时表头全选包含其他页数据Forse già presa @dvd233 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Tencent/tdesign-react#4416 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
area:widget bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
interledger/publisher-tools#894 ·
I maintainer di solito rispondono entro 1 giorno