Skip to content

Commit 1dcc870

Browse files
snowystingersnowystinger
authored
Update Picker placeholder to be shorter (#6796)
1 parent 7aadd66 commit 1dcc870

File tree

2 files changed

+26
-26
lines changed

2 files changed

+26
-26
lines changed

‎packages/@react-spectrum/picker/intl/en-US.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
{
2-
"placeholder": "Select an option…",
2+
"placeholder": "Select…",
33
"loading": "Loading…"
44
}

‎packages/@react-spectrum/picker/test/Picker.test.js‎

Lines changed: 25 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ describe('Picker', function () {
7070
expect(picker).toHaveAttribute('data-testid', 'test');
7171

7272
let label = getAllByText('Test')[0];
73-
let value = getByText('Select an option…');
73+
let value = getByText('Select…');
7474
expect(label).toBeVisible();
7575
expect(value).toBeVisible();
7676
});
@@ -362,7 +362,7 @@ describe('Picker', function () {
362362
expect(listbox).toBeVisible();
363363
expect(onOpenChange).not.toBeCalled();
364364

365-
let picker = getByLabelText('Select an option…');
365+
let picker = getByLabelText('Select…');
366366
expect(picker).toHaveAttribute('aria-expanded', 'true');
367367
expect(picker).toHaveAttribute('aria-controls', listbox.id);
368368

@@ -393,7 +393,7 @@ describe('Picker', function () {
393393
expect(listbox).toBeVisible();
394394
expect(onOpenChange).not.toBeCalled();
395395

396-
let picker = getByLabelText('Select an option…');
396+
let picker = getByLabelText('Select…');
397397
expect(picker).toHaveAttribute('aria-expanded', 'true');
398398
expect(picker).toHaveAttribute('aria-controls', listbox.id);
399399

@@ -721,7 +721,7 @@ describe('Picker', function () {
721721
expect(listbox).toBeVisible();
722722
expect(onOpenChange).not.toBeCalled();
723723

724-
let picker = getByLabelText('Select an option…');
724+
let picker = getByLabelText('Select…');
725725
expect(picker).toHaveAttribute('aria-expanded', 'true');
726726
expect(picker).toHaveAttribute('aria-controls', listbox.id);
727727

@@ -751,7 +751,7 @@ describe('Picker', function () {
751751
expect(getByRole('listbox')).toBeVisible();
752752
expect(onOpenChange).not.toBeCalled();
753753

754-
let picker = getByLabelText('Select an option…');
754+
let picker = getByLabelText('Select…');
755755
expect(picker).toHaveAttribute('aria-expanded', 'true');
756756

757757
let listbox = getByRole('listbox');
@@ -804,7 +804,7 @@ describe('Picker', function () {
804804
expect(picker).toHaveAttribute('aria-haspopup', 'listbox');
805805

806806
let label = getAllByText('Test')[0];
807-
let value = getByText('Select an option…');
807+
let value = getByText('Select…');
808808
expect(label).toHaveAttribute('id');
809809
expect(value).toHaveAttribute('id');
810810
expect(picker).toHaveAttribute('aria-labelledby', `${value.id} ${label.id}`);
@@ -829,7 +829,7 @@ describe('Picker', function () {
829829
);
830830

831831
let picker = getByRole('button');
832-
let value = getByText('Select an option…');
832+
let value = getByText('Select…');
833833
expect(picker).toHaveAttribute('id');
834834
expect(value).toHaveAttribute('id');
835835
expect(picker).toHaveAttribute('aria-label', 'Test');
@@ -855,7 +855,7 @@ describe('Picker', function () {
855855
);
856856

857857
let picker = getByRole('button');
858-
let value = getByText('Select an option…');
858+
let value = getByText('Select…');
859859
expect(picker).toHaveAttribute('id');
860860
expect(value).toHaveAttribute('id');
861861
expect(picker).toHaveAttribute('aria-labelledby', `${value.id} foo`);
@@ -880,7 +880,7 @@ describe('Picker', function () {
880880
);
881881

882882
let picker = getByRole('button');
883-
let value = getByText('Select an option…');
883+
let value = getByText('Select…');
884884
expect(picker).toHaveAttribute('id');
885885
expect(value).toHaveAttribute('id');
886886
expect(picker).toHaveAttribute('aria-label', 'Test');
@@ -914,7 +914,7 @@ describe('Picker', function () {
914914
expect(span).not.toHaveAttribute('aria-hidden');
915915

916916
let label = span.parentElement;
917-
let value = getByText('Select an option…');
917+
let value = getByText('Select…');
918918
expect(label).toHaveAttribute('id');
919919
expect(value).toHaveAttribute('id');
920920
expect(picker).toHaveAttribute('aria-labelledby', `${value.id} ${label.id}`);
@@ -978,7 +978,7 @@ describe('Picker', function () {
978978
);
979979

980980
let picker = getByRole('button');
981-
expect(picker).toHaveTextContent('Select an option…');
981+
expect(picker).toHaveTextContent('Select…');
982982
await user.click(picker);
983983
act(() => jest.runAllTimers());
984984

@@ -1015,7 +1015,7 @@ describe('Picker', function () {
10151015
);
10161016

10171017
let picker = getByRole('button');
1018-
expect(picker).toHaveTextContent('Select an option…');
1018+
expect(picker).toHaveTextContent('Select…');
10191019
await user.click(picker);
10201020
act(() => jest.runAllTimers());
10211021

@@ -1086,7 +1086,7 @@ describe('Picker', function () {
10861086
);
10871087

10881088
let picker = getByRole('button');
1089-
expect(picker).toHaveTextContent('Select an option…');
1089+
expect(picker).toHaveTextContent('Select…');
10901090
await user.click(picker);
10911091
act(() => jest.runAllTimers());
10921092

@@ -1132,7 +1132,7 @@ describe('Picker', function () {
11321132
);
11331133

11341134
let picker = getByRole('button');
1135-
expect(picker).toHaveTextContent('Select an option…');
1135+
expect(picker).toHaveTextContent('Select…');
11361136
act(() => {picker.focus();});
11371137

11381138
fireEvent.keyDown(picker, {key: 'ArrowUp'});
@@ -1177,7 +1177,7 @@ describe('Picker', function () {
11771177
);
11781178

11791179
let picker = getByRole('button');
1180-
expect(picker).toHaveTextContent('Select an option…');
1180+
expect(picker).toHaveTextContent('Select…');
11811181
await user.click(picker);
11821182
act(() => jest.runAllTimers());
11831183

@@ -1223,7 +1223,7 @@ describe('Picker', function () {
12231223
);
12241224

12251225
let picker = getByRole('button');
1226-
expect(picker).toHaveTextContent('Select an option…');
1226+
expect(picker).toHaveTextContent('Select…');
12271227
expect(onOpenChangeSpy).toHaveBeenCalledTimes(0);
12281228
await user.click(picker);
12291229
act(() => jest.runAllTimers());
@@ -1370,7 +1370,7 @@ describe('Picker', function () {
13701370
);
13711371

13721372
let picker = getByRole('button');
1373-
expect(picker).toHaveTextContent('Select an option…');
1373+
expect(picker).toHaveTextContent('Select…');
13741374
await user.click(picker);
13751375
act(() => jest.runAllTimers());
13761376

@@ -1443,7 +1443,7 @@ describe('Picker', function () {
14431443
);
14441444

14451445
let picker = getByRole('button');
1446-
expect(picker).toHaveTextContent('Select an option…');
1446+
expect(picker).toHaveTextContent('Select…');
14471447
await user.click(picker);
14481448
act(() => jest.runAllTimers());
14491449

@@ -1545,7 +1545,7 @@ describe('Picker', function () {
15451545

15461546
let picker = getByRole('button');
15471547
act(() => {picker.focus();});
1548-
expect(picker).toHaveTextContent('Select an option…');
1548+
expect(picker).toHaveTextContent('Select…');
15491549
fireEvent.keyDown(picker, {key: 'ArrowDown'});
15501550
act(() => jest.runAllTimers());
15511551

@@ -1641,7 +1641,7 @@ describe('Picker', function () {
16411641

16421642
let picker = getByRole('button');
16431643
await user.tab();
1644-
expect(picker).toHaveTextContent('Select an option…');
1644+
expect(picker).toHaveTextContent('Select…');
16451645
await user.keyboard('{ArrowLeft}');
16461646
act(() => jest.runAllTimers());
16471647
expect(onSelectionChange).toHaveBeenCalledTimes(1);
@@ -1687,7 +1687,7 @@ describe('Picker', function () {
16871687

16881688
let picker = getByRole('button');
16891689
act(() => {picker.focus();});
1690-
expect(picker).toHaveTextContent('Select an option…');
1690+
expect(picker).toHaveTextContent('Select…');
16911691

16921692
fireEvent.keyDown(picker, {key: 't'});
16931693
fireEvent.keyUp(picker, {key: 't'});
@@ -1715,7 +1715,7 @@ describe('Picker', function () {
17151715

17161716
let picker = getByRole('button');
17171717
act(() => {picker.focus();});
1718-
expect(picker).toHaveTextContent('Select an option…');
1718+
expect(picker).toHaveTextContent('Select…');
17191719

17201720
fireEvent.keyDown(picker, {key: 't'});
17211721
fireEvent.keyUp(picker, {key: 't'});
@@ -1744,7 +1744,7 @@ describe('Picker', function () {
17441744

17451745
let picker = getByRole('button');
17461746
act(() => {picker.focus();});
1747-
expect(picker).toHaveTextContent('Select an option…');
1747+
expect(picker).toHaveTextContent('Select…');
17481748

17491749
fireEvent.keyDown(picker, {key: 't'});
17501750
fireEvent.keyUp(picker, {key: 't'});
@@ -1772,7 +1772,7 @@ describe('Picker', function () {
17721772
);
17731773

17741774
let picker = getByRole('button');
1775-
expect(picker).toHaveTextContent('Select an option…');
1775+
expect(picker).toHaveTextContent('Select…');
17761776

17771777
let hiddenLabel = getByText('Test', {hidden: true, selector: 'label'});
17781778
expect(hiddenLabel.tagName).toBe('LABEL');
@@ -2500,7 +2500,7 @@ describe('Picker', function () {
25002500
jest.runAllTimers();
25012501
});
25022502

2503-
expect(button).toHaveTextContent('Select an option…');
2503+
expect(button).toHaveTextContent('Select…');
25042504
expect(listbox).not.toBeInTheDocument();
25052505
expect(onClick.mock.calls[0][0].target).toBeInstanceOf(HTMLAnchorElement);
25062506
expect(onClick.mock.calls[0][0].target.href).toBe('https://google.com/');

0 commit comments

Comments
 (0)