[jss-compose] When composing from a class that takes props, the inheriting class needs props also

Beschreibung

const useStyles = makeStyles({
  icon: props => ({ // <---------- base class
    fontSize: Number(props.size)
  }),
  successIcon: props => ({
    composes: '$icon', // This works, because it takes props, even though not using them
    color: 'green'
  }),
  failIcon: {
    composes: '$icon',  // This extension should work, but it doesn't
    color: 'red'
  }
})

Contributor Guide

Tech Stack

javascriptreactcss

Domain

frontend

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.

1-3 hours

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

JavaScript basicsJSS composition concept

EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.

70

Research-Richtung

Examine the jss compose plugin implementation, specifically how it resolves composed classes when the base class is a function (with props) and the extending class is an object. Look at the test files for jss compose to understand expected behavior. The issue likely requires adding a check to handle the case where the extending class does not need props but the base does. A fix might involve ensuring that when composing, if the base is a function, the composed result should also be a function to preserve prop passing.