Merge pull request #6132 from BenJamesBen/test-view-optional-measurements
fix(shared): In Test view, include optional measurements
This commit is contained in:
commit
409b183373
1 changed files with 11 additions and 2 deletions
|
@ -31,6 +31,15 @@ export const TestMenu = ({ design, patternConfig, settings, update }) => {
|
|||
const { t } = useTranslation(ns)
|
||||
|
||||
const allOptions = flattenOptions(patternConfig.options, settings)
|
||||
const allMeasurements = patternConfig.measurements
|
||||
.concat(patternConfig.optionalMeasurements)
|
||||
.sort((a, b) => {
|
||||
const ta = t(`measurements:${a}`)
|
||||
const tb = t(`measurements:${b}`)
|
||||
if (ta < tb) return -1
|
||||
else if (ta > tb) return 1
|
||||
else return 0
|
||||
})
|
||||
|
||||
return (
|
||||
<Accordion
|
||||
|
@ -76,8 +85,8 @@ export const TestMenu = ({ design, patternConfig, settings, update }) => {
|
|||
</Fragment>,
|
||||
<ListInput
|
||||
key="b"
|
||||
list={patternConfig.measurements.map((m) => ({
|
||||
label: t(m),
|
||||
list={allMeasurements.map((m) => ({
|
||||
label: t(`measurements:${m}`),
|
||||
val: m,
|
||||
}))}
|
||||
update={(value) => {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue