AllenFang/react-bootstrap-table

Extend filters to allow html attributes to be passed down

Open

#1715 aperta il 26 ott 2017

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (795 fork)batch import
help wantedin progress

Metriche repository

Star
 (2240 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Hello. Currently it appears there is no way to add an html attribute to the underlying markup that is generated by the built-in filters such as <SelectFilter>.

My use-case would be adding an id, data, or aria attribute to the html elements such as a <select>.

Attributes are passed to the component in the JSX. https://github.com/AllenFang/react-bootstrap-table/blob/281db991c08f1d4cad2c522b13875a97d2e27e0f/src/TableHeaderColumn.js#L70

But they aren't shared with the HTML elements. https://github.com/AllenFang/react-bootstrap-table/blob/281db991c08f1d4cad2c522b13875a97d2e27e0f/src/filters/Select.js#L98-L104

I'm happy to submit a PR if you agree with the use.

Guida contributor

Direzione di ricerca
Indaga i file sorgente: src/TableHeaderColumn.js e src/filters/Select.js. In Select.js, il componente attualmente renderizza un elemento <select> con props specifici; aggiungi una prop come `htmlAttributes` o semplicemente spalma props aggiuntivi sul <select>. Assicurati di aggiornare i test. Controlla anche se filtri simili necessitano dello stesso trattamento. Poiché l'autore ha offerto una PR, coordina con i manutentori per una guida.
Tech stack
javascriptreact
Dominio
frontendaccessibility
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Basic ReactProps passing in React
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80