Allow ReactiveList to optionally render a sort label when using sortOptions
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- javascript, react
- Ambito
- frontend
Direzione di ricerca
Inizia individuando l’implementazione di ReactiveList di sortOptions e renderSortOptions, quindi confronta la relativa prop renderResultStats e il comportamento di innerClass. Determina se la label debba usare una render prop personalizzata o un’opzione showSortLabel predefinita e definisci il comportamento di rendering previsto prima di verificare i test React pertinenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- JavaScript
- Stelle
- 4.9k
- Fork
- 479
- Merge medio
- 4g 10h
- PR unite (30g)
- 4
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di appbaseio/reactivesearch
-
ci: add Node.js version matrix, caching, and lint step to test workflowForse già presa @Talha12Shiekh l’ha presa 71 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
appbaseio/reactivesearch#2322 ·
I maintainer di solito rispondono entro 1 giorno
-
chore: add Dependabot config for automated dependency updatesForse già presa @Talha12Shiekh l’ha presa 71 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
appbaseio/reactivesearch#2321 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
security: user-controlled URL params injected directly into Redux store via JSON.parseForse già presa @Talha12Shiekh l’ha presa 71 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
appbaseio/reactivesearch#2319 ·
I maintainer di solito rispondono entro 1 giorno
-
refactor: consolidate 12 near-identical SVG icon components into one patternForse già presa @Talha12Shiekh l’ha presa 73 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
appbaseio/reactivesearch#2324 ·
I maintainer di solito rispondono entro 1 giorno
-
refactor: SearchBox.js is 2692 lines — extract subcomponents for maintainabilityForse già presa @Talha12Shiekh l’ha presa 71 giorni fa. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
appbaseio/reactivesearch#2323 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di appbaseio/reactivesearch
Issue simili
-
[BUG] Multi-day events show "Ended" while still in progressForse già presa @tarunagnihotri534 l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
data-umbrella/du-event-board#231 · 2 commenti ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
NaturalIntelligence/fast-xml-parser#888 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 80/100
USACE/chart-docs#766 ·
-
bug callouts regression revealjs
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
quarto-dev/quarto-cli#15014 ·
I maintainer di solito rispondono entro 1 giorno