Add an option to change the highlighting background color of `updatemenus`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 18.3k
- Forks
- 2k
- Ø Merge
- 2 T. 12 Std.
- Gemergte PRs (30 T.)
- 28
Beschreibung
Currently only the normal (i. e. unhighlighted) background color can be changed through the parameter bgcolor. If you set this background to a dark coloring, you most probaby also want to adjust the text font to a bright coloring so that the text is still readable. But then the text becomes pretty unreadable while being highlighted since the highlighting background color is very bright!
Therefore I think an additional parameter bgcolor_active or the like should be introduced.
Additionally, an even more sophisticated solution could involve an automatic adjustment of the text font color and/or background color depending on the color brightness of the user defined bgcolor and/or font.color values (this seems to be already the case with the font color of hoverinfo labels).
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit den Referenzeinträgen von layout updatemenus für bgcolor und font.color und verfolge anschließend die bestehende Behandlung des hervorgehobenen Zustands. Entscheide, ob die angeforderte Option für den aktiven Hintergrund die klar abgegrenzte Änderung ist; fertig bedeutet, dass die Option dokumentiert ist und hervorgehobene Steuerelemente bei dunklen normalen Hintergründen lesbar bleiben, mit Testabdeckung für das neue Verhalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- data-visualization
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100