Skip to content

Commit

Permalink
4109 - ODP edit: Methods used MultiSelect refactor (#4110)
Browse files Browse the repository at this point in the history
  • Loading branch information
minotogna authored Nov 13, 2024
1 parent 1028acc commit edbccb6
Show file tree
Hide file tree
Showing 8 changed files with 8 additions and 216 deletions.
81 changes: 0 additions & 81 deletions src/client/components/MultiSelect/MultiSelect.scss

This file was deleted.

99 changes: 0 additions & 99 deletions src/client/components/MultiSelect/MultiSelect.tsx

This file was deleted.

1 change: 0 additions & 1 deletion src/client/components/MultiSelect/index.ts

This file was deleted.

5 changes: 0 additions & 5 deletions src/client/components/MultiSelect/option.ts

This file was deleted.

13 changes: 0 additions & 13 deletions src/client/pages/OriginalDataPoint/OriginalDataPointSection.scss
Original file line number Diff line number Diff line change
Expand Up @@ -22,19 +22,6 @@
// custom data grid
.odp__section {
.data-grid {
// TODO: multi select refactor style
.multi-select__closed-content {
padding: $spacing-xxs;
}

// TODO: multi select refactor style
.multi-select__opened {
border-radius: unset;
top: -1px;
left: -1px;
width: calc(100% + 2px);
}

svg.icon-reference-files {
stroke: none;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@ import { ODPDataSourceMethod, OriginalDataPoint } from 'meta/assessment/original
import { Topics } from 'meta/messageCenter'

import { DataCell, DataRow, DataRowAction, DataRowActionType } from 'client/components/DataGrid'
import MultiSelect from 'client/components/MultiSelect'
import { Option } from 'client/components/MultiSelect/option'
import Select, { Option } from 'client/components/Inputs/Select'
import { useShowReviewIndicator } from 'client/pages/OriginalDataPoint/hooks/useShowReviewIndicator'

import { useIsDisabled } from '../hooks/useIsDisabled'
Expand Down Expand Up @@ -59,11 +58,12 @@ const MethodsUsed: React.FC<Props> = (props: Props) => {
<DataRow actions={actions}>
<DataCell header>{t('nationalDataPoint.methodsUsed')}</DataCell>
<DataCell lastCol>
<MultiSelect
<Select
disabled={disabled}
isMulti
onChange={onChange}
options={options}
values={originalDataPoint.dataSourceMethods ?? []}
value={originalDataPoint.dataSourceMethods ?? []}
/>
</DataCell>
</DataRow>
Expand Down
4 changes: 0 additions & 4 deletions src/client/pages/Print/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -49,10 +49,6 @@
padding: 10px 0;
}

.multi-select__placeholder {
display: none;
}

.assessment-section__table-title {
margin-bottom: 16px;
}
Expand Down
13 changes: 4 additions & 9 deletions src/client/styles/rtl.scss
Original file line number Diff line number Diff line change
Expand Up @@ -97,11 +97,6 @@
left: 24px;
}

.multi-select__opened-item-label {
margin-right: 6px;
margin-left: initial;
}

.edit-user__form-label {
padding-right: initial;
padding-left: 14px;
Expand Down Expand Up @@ -159,10 +154,10 @@
margin-right: 4px;
}

.multi-select,
.multi-select:not([disabled]) {
background-position: left 10px top 50%;
}
//.multi-select,
//.multi-select:not([disabled]) {
// background-position: left 10px top 50%;
//}

.autosave-status {
right: initial;
Expand Down

0 comments on commit edbccb6

Please sign in to comment.