cssinjs/jss

TS error using a function to customise defaults with the default unit plugin

Open

#1.444 geöffnet am 20. Jan. 2021

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (7.000 Stars) (434 Forks)batch import
help wantedtypescript

Beschreibung

Expected behavior: Use the 'jss-plugin-default-unit' to override default units for numeric values in combination with TypeScript.

Describe the bug: The plugin itself doesn't appear to have any issues but I am receiving TypeScript related errors in my IDE when using a function to define an option for the plugin.

const options = {
    width: val => `${val / 2}px`
}
jss.use(defaultUnit(options));

Source: https://cssinjs.org/jss-plugin-default-unit/?v=v10.5.0#customizing-defaults

I'm not sure if the following is the cause but the two definitions of the Options type do not seem to match:

Codesandbox: N/A

Versions:

  • react-jss: 10.5.0
  • typescript: 4.0.3
  • OS: Linux

Contributor Guide

Tech Stack
typescriptjavascriptcss
Domain
frontenddeveloper experience
Issue Type
bug
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.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
TypeScript typesJSS plugin system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85
Research-Richtung
Examine the two files: packages/jss plugin default unit/src/index.js and packages/jss plugin default unit/src/index.d.ts. The Options type in the .d.ts may need to accept a function type for default values. Check the JssStyle type and ensure consistency. No existing PRs or comments provide additional guidance.