Skip to content

Commit d651470

Browse files
experimental->next
1 parent 773698b commit d651470

File tree

2 files changed

+12
-12
lines changed

2 files changed

+12
-12
lines changed

src/rules/__tests__/a11y-use-next-tooltip.test.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -13,38 +13,38 @@ const ruleTester = new RuleTester({
1313

1414
ruleTester.run('a11y-use-next-tooltip', rule, {
1515
valid: [
16-
`import {Tooltip} from '@primer/react/experimental';`,
16+
`import {Tooltip} from '@primer/react/next';`,
1717
`import {UnderlineNav, Button} from '@primer/react';
18-
import {Tooltip} from '@primer/react/experimental';`,
18+
import {Tooltip} from '@primer/react/next';`,
1919
`import {UnderlineNav, Button} from '@primer/react';
20-
import {Tooltip, SelectPanel} from '@primer/react/experimental';`,
20+
import {Tooltip, SelectPanel} from '@primer/react/next';`,
2121
],
2222
invalid: [
2323
{
2424
code: `import {Tooltip} from '@primer/react';`,
2525
errors: [{messageId: 'useNextTooltip'}],
26-
output: `import {Tooltip} from '@primer/react/experimental';`,
26+
output: `import {Tooltip} from '@primer/react/next';`,
2727
},
2828
{
2929
code: `import {Tooltip, Button} from '@primer/react';\n<Tooltip aria-label="tooltip text"><Button>Button</Button></Tooltip>`,
3030
errors: [{messageId: 'useNextTooltip'}, {messageId: 'useTextProp'}],
31-
output: `import {Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/experimental';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
31+
output: `import {Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/next';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
3232
},
3333
{
3434
code: `import {ActionList, ActionMenu, Tooltip, Button} from '@primer/react';\n<Tooltip aria-label="tooltip text"><Button>Button</Button></Tooltip>`,
3535
errors: [
3636
{messageId: 'useNextTooltip', line: 1},
3737
{messageId: 'useTextProp', line: 2},
3838
],
39-
output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/experimental';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
39+
output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/next';\n<Tooltip text="tooltip text"><Button>Button</Button></Tooltip>`,
4040
},
4141
{
4242
code: `import {ActionList, ActionMenu, Button, Tooltip} from '@primer/react';\n<Tooltip aria-label="tooltip text"><Button aria-label="test">Button</Button></Tooltip>`,
4343
errors: [
4444
{messageId: 'useNextTooltip', line: 1},
4545
{messageId: 'useTextProp', line: 2},
4646
],
47-
output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/experimental';\n<Tooltip text="tooltip text"><Button aria-label="test">Button</Button></Tooltip>`,
47+
output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/next';\n<Tooltip text="tooltip text"><Button aria-label="test">Button</Button></Tooltip>`,
4848
},
4949
{
5050
code: `import {ActionList, ActionMenu, Tooltip, Button} from '@primer/react';\n<Tooltip aria-label="tooltip text" noDelay={true} wrap={true} align="left"><Button>Button</Button></Tooltip>`,
@@ -55,7 +55,7 @@ ruleTester.run('a11y-use-next-tooltip', rule, {
5555
{messageId: 'wrapRemoved', line: 2},
5656
{messageId: 'alignRemoved', line: 2},
5757
],
58-
output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/experimental';\n<Tooltip text="tooltip text" ><Button>Button</Button></Tooltip>`,
58+
output: `import {ActionList, ActionMenu, Button} from '@primer/react';\nimport {Tooltip} from '@primer/react/next';\n<Tooltip text="tooltip text" ><Button>Button</Button></Tooltip>`,
5959
},
6060
],
6161
})

src/rules/a11y-use-next-tooltip.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,14 @@ module.exports = {
66
meta: {
77
type: 'suggestion',
88
docs: {
9-
description: 'recommends the use of @primer/react/experimental Tooltip component',
9+
description: 'recommends the use of @primer/react/next Tooltip component',
1010
category: 'Best Practices',
1111
recommended: true,
1212
},
1313
fixable: true,
1414
schema: [],
1515
messages: {
16-
useNextTooltip: 'Please use @primer/react/experimental Tooltip component that has accessibility improvements',
16+
useNextTooltip: 'Please use @primer/react/next Tooltip component that has accessibility improvements',
1717
useTextProp: 'Please use the text prop instead of aria-label',
1818
noDelayRemoved: 'noDelay prop is removed. Tooltip now has no delay by default',
1919
wrapRemoved: 'wrap prop is removed. Tooltip now wraps by default',
@@ -40,7 +40,7 @@ module.exports = {
4040
fix(fixer) {
4141
// If Tooltip is the only import, replace the whole import statement
4242
if (!hasOtherImports) {
43-
return fixer.replaceText(node.source, `'@primer/react/experimental'`)
43+
return fixer.replaceText(node.source, `'@primer/react/next'`)
4444
} else {
4545
// Otherwise, remove Tooltip from the import statement and add a new import statement with the correct path
4646
const tooltipSpecifier = node.specifiers.find(
@@ -68,7 +68,7 @@ module.exports = {
6868
fixer.removeRange([start, end]),
6969
fixer.insertTextAfterRange(
7070
[node.range[1], node.range[1]],
71-
`\nimport {Tooltip} from '@primer/react/experimental';`,
71+
`\nimport {Tooltip} from '@primer/react/next';`,
7272
),
7373
]
7474
}

0 commit comments

Comments
 (0)