cssinjs/jss

Extend function property values are converted to strings

Open

#1.134 geöffnet am 17. Juni 2019

Auf GitHub ansehen
 (0 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)JavaScript (434 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (7.000 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Expected behavior: A clear and concise description of what you expected to happen.

const styles = {
  linkCommon: {
    color: () => "orange"
  },
  link: {
    extend: "linkCommon"
  }
};

The link should be orange.

Describe the bug: A clear and concise description of what happend instead of the expected behavior.

The function is being converted to a string before being inserted into the DOM.

image

Codesandbox link:

Playground: https://codesandbox.io/embed/core-jss-playground-ef0j4

Versions (please complete the following information):

  • jss: 10 alpha 18
  • jss-preset-default: 10 alpha 18
  • Browser Chrome 75
  • OS: OX X

Feel free to add any additional versions which you may think are relevant to the bug.

Not to be confused with https://github.com/cssinjs/jss/issues/479

Contributor Guide

Research-Richtung
Untersuche, wie die `extend` Eigenschaft Stile zusammenführt, wenn Werte Funktionen sind. Sieh dir die Quelldateien im JSS Repository an, insbesondere die Logik zur Stilzusammenführung (z.B. `src/`). Das Problem zeigt, dass wenn eine Funktion als Wert in einem erweiterten Stil verwendet wird, sie in einen String umgewandelt wird. Vergleiche dies mit der Behandlung von Funktionen in nicht erweiterten Stilen. Prüfe, ob es einen Testfall für ähnliche Szenarien gibt, und erwäge, einen hinzuzufügen. Der Playground Link deutet darauf hin, dass der Code jss preset default verwendet; stelle sicher, dass die Korrektur mit dem Standard Preset funktioniert.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75