gomorizsolt / gomorizsolt/react-theme-toggle-button

Add option to invert logic of the button icons

Offen
#5 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
good first issue
Vorherrschende Sprache
JavaScript
Sterne
0
Forks
1
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

I'd like to have the sun displayed if the light theme is active, so the sun will show to indicate that it's the light theme and if the dark theme is active I'd like to show the moon.

The current implementation is showing what the button will do but the described behaviour feels more natural IMO. Both could be added so it can be changed with a prop.

To make it more clear, I've created a [sandbox](https://codesandbox.io/s/react-theme-toggle-button-olr5g) with the modified button code.

If you like I could create a PR for this.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Beginne damit, die bestehende Button-Implementierung zu überprüfen und sie mit dem geänderten Button-Code in der verlinkten CodeSandbox zu vergleichen. Ermittle, wie die Komponente derzeit das Icon auswählt und wo ein Prop den Anzeigemodus steuern könnte. Als abgeschlossen gilt die Aufgabe, wenn das helle Theme eine Sonne anzeigt, das dunkle Theme einen Mond anzeigt und das bestehende Verhalten weiterhin verfügbar ist, wenn es ausgewählt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, react
Bereich
frontend
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.