react-bootstrap-table/react-bootstrap-table2

defaultSorted cannot sort more than one column

Open

#1,083 opened on 2019年9月3日

GitHub で見る
 (4 comments) (2 reactions) (1 assignee)JavaScript (1,249 stars) (415 forks)batch import
help wantedsort

説明

Describe the bug Only the first column of the defaultSorted array is considered. I see #717 was closed, but neither the documentation nor the linked example contains two columns in defaultSorted.

codesandbox https://codesandbox.io/embed/react-bootstrap-table-next-basic-example-69nq0

Note that the second sort column is on price descending, so the row with id 2 should be displayed first.

In general, the expected behavior here is that the table is first sorted by name, and then for items with the same name, they should be ordered by price in descending.

Happy to provide more examples as needed.

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

技術スタック
reactjavascript
領域
frontend
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
ReactJavaScriptreact bootstrap table2 basics
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
First, reproduce the issue using the provided CodeSandbox (inline note: the second sort column on 'price' descending is not applied). Examine the source code for the `defaultSorted` prop handling, likely in the sorting utility or in the main table component. The referenced issue #717 was closed but may reveal previous attempts. Check if the sorting logic iterates only over the first entry of `defaultSorted` and needs to handle multiple columns by chaining sort comparators. Look at the `src/sorting.js` or similar file for the sort state initialization and applyChanges methods.