AllenFang/react-bootstrap-table

Extend filters to allow html attributes to be passed down

Open

#1,715 opened on 2017年10月26日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)JavaScript (2,240 stars) (795 forks)batch import
help wantedin progress

説明

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.

コントリビューターガイド

技術スタック
javascriptreact
領域
frontendaccessibility
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Basic ReactProps passing in React
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
80
調査方針
Investigate the source files: src/TableHeaderColumn.js and src/filters/Select.js. In Select.js, the component currently renders a <select> element with specific props; add a prop like `htmlAttributes` or simply spread additional props onto the <select>. Ensure that tests are updated. Also check if similar filters need the same treatment. Since the author offered a PR, coordinate with maintainers for guidance.