fix(website): pass order property to download dialog field selector - #6179
Conversation
The download dialog's field selector was missing the `order` property when constructing FieldItem arrays, causing fields and sections to appear in arbitrary order instead of matching the configured order used by the search form and customize columns modals. Closes #6178 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Claude finished @anna-parker's task in 36s —— View job Code Review
The fix is correct and minimal. The common The one-line addition resolves both the per-group field ordering and the inter-group header ordering. No other callers of this component are affected since this is the download dialog's own wrapper. No issues found. LGTM. |
|
doesn't matter for this one, but for any slightly bigger PRs we should generally keep to our policy of having two human eyes see it |
Summary
The download dialog's field selector was missing the
orderproperty when constructingFieldItem[]arrays from schema metadata. This caused fields and section groups to appear in arbitrary order (based on insertion order) instead of matching the configured order used by both the search form and customize columns modals.The fix adds
order: field.orderto the field item mapping inwebsite/src/components/SearchPage/DownloadDialog/FieldSelector/FieldSelectorModal.tsx, ensuring all three field selector modals display metadata in the same consistent order.Closes #6178
Test plan
🤖 Generated with Claude Code
🚀 Preview: https://fix-download-field-order.loculus.org