Allow ReactiveList to optionally render a sort label when using sortOptions
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Fonctionnalité
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- javascript, react
- Domaine
- frontend
Piste de recherche
Commencez par localiser l’implémentation de ReactiveList de sortOptions et renderSortOptions, puis comparez son prop renderResultStats et le comportement de innerClass. Déterminez si le label doit utiliser un render prop personnalisé ou une option showSortLabel par défaut, et définissez le comportement de rendu attendu avant de vérifier les tests React pertinents.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Affected Projects
React
Is your feature request related to a problem? Please describe.
In our current application we make use of the ReactiveList sorting however this only produces the select dropdown and has no way to tap into the render process to add a custom label (i.e. Material UI Typography) named "Sort By".
Describe the solution you'd like
A few options would work. Simple option is to add a renderSortLabel prop which render the custom label and this is then referenced in the renderSortOptions function and if set renders the label before the select.
Or have a prop named showSortLabel that renders a default "Sort By" label.
Alternative option could be to replicate that of the renderResultStats function where a custom render function is passed in as a prop and would handles the complete generation of the sort although this does is make it a lot more complex due to the handleSortChange handler.
Describe alternatives you've considered
Will start looking the innerClass to see if it is possible to tap into the sortOptions this and use the ::before pseudo-element.
Additional context
- Langage dominant
- JavaScript
- Étoiles
- 4.9k
- Forks
- 478
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de appbaseio/reactivesearch
-
ci: add Node.js version matrix, caching, and lint step to test workflowPeut-être pris @Talha12Shiekh l’a pris il y a 94 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
appbaseio/reactivesearch#2322 ·
-
chore: add Dependabot config for automated dependency updatesPeut-être pris @Talha12Shiekh l’a pris il y a 94 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
appbaseio/reactivesearch#2321 · 1 commentaire ·
-
security: user-controlled URL params injected directly into Redux store via JSON.parsePeut-être pris @Talha12Shiekh l’a pris il y a 94 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
appbaseio/reactivesearch#2319 ·
-
refactor: consolidate 12 near-identical SVG icon components into one patternPeut-être pris @Talha12Shiekh l’a pris il y a 96 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
appbaseio/reactivesearch#2324 ·
-
refactor: SearchBox.js is 2692 lines — extract subcomponents for maintainabilityPeut-être pris @Talha12Shiekh l’a pris il y a 94 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
appbaseio/reactivesearch#2323 ·
Toutes les issues de appbaseio/reactivesearch
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Les mainteneurs répondent en général sous 13 jours
-
Slack: DM thread replies sent with "Also send to..." never reach the agentPeut-être pris @roboeve l’a pris aujourd’hui. Ouvertebug channel:slack p2
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
vercel/eve#5774 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Android: keyboard height remains stale when IME disappears without a closing animationPeut-être pris @jaxnnux l’a pris aujourd’hui. OuverteMissing info Missing repro Platform: Android
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
software-mansion/react-native-reanimated#11106 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement version-minor
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
cuttle-cards/cuttle#1467 ·
Les mainteneurs répondent en général sous 1 jour
-
2026-11 bug good first issue gsoc-2027 project: mockuper
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
alphaonelabs/website#1147 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour