Skip to content

Commit dd5f5e8

Browse files
authored
Add 'Level-2 provider' label to provider dropdown (#437)
* Add 'Level-2 provider' label to provider dropdown - Added InputLabel component to the level-2 provider dropdown - Label appears in both Create Provider and Edit Provider forms - Improves UI clarity for users selecting level-2 providers * Fix Level-2 provider label display - Add id prop to InputLabel for proper Material-UI linking - Add labelId and label props to Select component - Ensures label displays correctly with outlined variant styling - Follows Material-UI pattern used elsewhere in the codebase * Remove displayEmpty to fix label overlap issue - Changed value from null to empty string for empty state - Removed displayEmpty prop which caused label overlap - Changed MenuItem value from null to empty string - Label now displays correctly above the select field
1 parent d1f8de2 commit dd5f5e8

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/modals/ModalEditProvider.jsx

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ import { Dialog, DialogTitle, DialogContent, DialogActions } from '@mui/material
1919
import withAuth from 'utils/withAuth';
2020
import TextField from '@mui/material/TextField';
2121
import { Checkbox, FormControlLabel } from '@mui/material';
22-
import { FormControl, Select, MenuItem } from '@mui/material';
22+
import { FormControl, Select, MenuItem, InputLabel } from '@mui/material';
2323
import Button from '@mui/material/Button';
2424
import { connect } from 'react-redux';
2525
import SuppliersActions from 'actions/SuppliersActions';
@@ -289,13 +289,15 @@ class ModalEditProvider extends Component {
289289
</div>
290290
<div style={rowStyle}>
291291
<FormControl style={{ flex: 1 }}>
292+
<InputLabel id="level2-provider-label">Level-2 provider</InputLabel>
292293
<Select
293-
value={this.state.form._migrateDataToProvider}
294+
labelId="level2-provider-label"
295+
label="Level-2 provider"
296+
value={this.state.form._migrateDataToProvider || ''}
294297
onChange={e => this.handleChange('_migrateDataToProvider', e.target.value)}
295-
displayEmpty
296298
>
297-
<MenuItem value={null} style={{ fontStyle: 'italic' }}>
298-
None
299+
<MenuItem value="">
300+
<em>None</em>
299301
</MenuItem>
300302
{providers
301303
.filter(provider => !provider.chouetteInfo.migrateDataToProvider)

0 commit comments

Comments
 (0)