Skip to content

Commit 0f0fd86

Browse files
committed
update aria labels
1 parent f902a06 commit 0f0fd86

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

web_ui/src/pages/project-details/components/project-models/train-model-dialog/advanced-settings/ui/range-parameter-field.component.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ export const RangeParameterField: FC<RangeParameterFieldProps> = ({
6868
};
6969

7070
return (
71-
<Flex gap={'size-100'}>
71+
<Flex gap={'size-100'} aria-label={name}>
7272
<NumberField
7373
isQuiet
7474
step={fieldStep}
@@ -90,7 +90,7 @@ export const RangeParameterField: FC<RangeParameterFieldProps> = ({
9090
step={fieldStep}
9191
flex={1}
9292
isDisabled={isDisabled}
93-
aria-label={`Change ${name} range value`}
93+
aria-label={`Change ${name} value`}
9494
UNSAFE_className={isDisabled ? '' : classes.rangeSlider}
9595
/>
9696
<NumberField

web_ui/src/pages/project-details/components/project-models/train-model-dialog/advanced-settings/ui/range-parameter-field.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ describe('RangeParameterField', () => {
3535

3636
expect(screen.getByLabelText(`Change ${name} start range value`)).toBeInTheDocument();
3737
expect(screen.getByLabelText(`Change ${name} end range value`)).toBeInTheDocument();
38-
expect(screen.getByLabelText(`Change ${name} range value`)).toBeInTheDocument();
38+
expect(screen.getByLabelText(`Change ${name} value`)).toBeInTheDocument();
3939
});
4040

4141
it('calls onChange when start value changes', async () => {
@@ -83,7 +83,7 @@ describe('RangeParameterField', () => {
8383
expect(screen.getByLabelText(`Change ${name} start range value`)).toBeDisabled();
8484
expect(screen.getByLabelText(`Change ${name} end range value`)).toBeDisabled();
8585

86-
const slider = screen.getByLabelText(`Change ${name} range value`);
86+
const slider = screen.getByLabelText(`Change ${name} value`);
8787

8888
expect(slider).toHaveClass('A-RCEa_is-disabled');
8989
});

0 commit comments

Comments
 (0)