Skip to content

Commit ac7f74a

Browse files
committed
upgrade packages and lint rules
1 parent cc6f83f commit ac7f74a

File tree

10 files changed

+3154
-1547
lines changed

10 files changed

+3154
-1547
lines changed

.size-snapshot.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,17 +2,17 @@
22
"dist/index.js": {
33
"bundled": 7471,
44
"minified": 7471,
5-
"gzipped": 2219
5+
"gzipped": 2246
66
},
77
"lib/cjs/index.js": {
88
"bundled": 14096,
99
"minified": 7791,
10-
"gzipped": 2148
10+
"gzipped": 2149
1111
},
1212
"lib/esm/index.js": {
1313
"bundled": 14044,
1414
"minified": 7758,
15-
"gzipped": 2144,
15+
"gzipped": 2145,
1616
"treeshaked": {
1717
"rollup": {
1818
"code": 7197,

README.md

Lines changed: 44 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -30,50 +30,54 @@ or
3030

3131
```jsx
3232
import React from 'react';
33-
import { render } from 'react-dom';
33+
import {render} from 'react-dom';
3434
import TooltipTrigger from 'react-popper-tooltip';
3535

36+
const Tooltip = ({
37+
getTooltipProps,
38+
getArrowProps,
39+
tooltipRef,
40+
arrowRef,
41+
placement
42+
}) => (
43+
<div
44+
{...getTooltipProps({
45+
ref: tooltipRef,
46+
className: 'tooltip-container'
47+
/* your props here */
48+
})}
49+
>
50+
<div
51+
{...getArrowProps({
52+
ref: arrowRef,
53+
'data-placement': placement,
54+
className: 'tooltip-arrow'
55+
/* your props here */
56+
})}
57+
/>
58+
Hello, World!
59+
</div>
60+
);
61+
62+
const Trigger = ({getTriggerProps, triggerRef}) => (
63+
<span
64+
{...getTriggerProps({
65+
ref: triggerRef,
66+
className: 'trigger'
67+
/* your props here */
68+
})}
69+
>
70+
Click Me!
71+
</span>
72+
);
73+
3674
render(
3775
<TooltipTrigger
3876
placement="right"
3977
trigger="click"
40-
tooltip={({
41-
getTooltipProps,
42-
getArrowProps,
43-
tooltipRef,
44-
arrowRef,
45-
placement
46-
}) => (
47-
<div
48-
{...getTooltipProps({
49-
ref: tooltipRef,
50-
className: 'tooltip-container'
51-
/* your props here */
52-
})}
53-
>
54-
<div
55-
{...getArrowProps({
56-
ref: arrowRef,
57-
'data-placement': placement,
58-
className: 'tooltip-arrow'
59-
/* your props here */
60-
})}
61-
/>
62-
Hello, World!
63-
</div>
64-
)}
78+
tooltip={Tooltip}
6579
>
66-
{({ getTriggerProps, triggerRef }) => (
67-
<span
68-
{...getTriggerProps({
69-
ref: triggerRef,
70-
className: 'trigger'
71-
/* your props here */
72-
})}
73-
>
74-
Click Me!
75-
</span>
76-
)}
80+
{Trigger}
7781
</TooltipTrigger>,
7882
document.getElementById('root')
7983
);
@@ -94,7 +98,7 @@ import React from 'react';
9498
import TooltipTrigger from 'react-popper-tooltip';
9599
import 'react-popper-tooltip/dist/styles.css';
96100

97-
const Tooltip = ({ tooltip, children, hideArrow ...props }) => (
101+
const Tooltip = ({tooltip, children, hideArrow ...props}) => (
98102
<TooltipTrigger
99103
{...props}
100104
tooltip={({
@@ -121,7 +125,7 @@ const Tooltip = ({ tooltip, children, hideArrow ...props }) => (
121125
</div>
122126
)}
123127
>
124-
{({ getTriggerProps, triggerRef }) => (
128+
{({getTriggerProps, triggerRef}) => (
125129
<span
126130
{...getTriggerProps({
127131
ref: triggerRef,
@@ -305,7 +309,7 @@ const tooltip = (
305309
These functions are used to apply props to the elements that you render.
306310
It's advisable to pass all your props to that function rather than applying them on the element
307311
yourself to avoid your props being overridden (or overriding the props returned). For example
308-
`<button {...getTriggerProps({ onClick: event => console.log(event))}>Click me</button>`
312+
`<button {...getTriggerProps({onClick: event => console.log(event))}>Click me</button>`
309313

310314
### children function
311315

babel.config.js

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,12 @@ module.exports = {
1010
'@babel/react'
1111
],
1212
plugins: [
13-
['@babel/proposal-class-properties', { loose: true }],
14-
['transform-react-remove-prop-types', { removeImport: true }]
15-
]
13+
['@babel/proposal-class-properties', {loose: true}],
14+
['transform-react-remove-prop-types', {removeImport: true}]
15+
],
16+
env: {
17+
test: {
18+
presets: ['@babel/env']
19+
}
20+
}
1621
};
Lines changed: 41 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -1,48 +1,46 @@
1-
import React, { memo } from 'react';
1+
import React, {memo} from 'react';
22
import TooltipTrigger from '../../src';
33
import '../../src/styles.css';
44

5-
const BasicTooltipTrigger = memo(
6-
({ tooltip, children, hideArrow, ...props }) => (
7-
<TooltipTrigger
8-
{...props}
9-
tooltip={({
10-
getTooltipProps,
11-
getArrowProps,
12-
tooltipRef,
13-
arrowRef,
14-
placement
15-
}) => (
16-
<span
17-
{...getTooltipProps({
18-
ref: tooltipRef,
19-
className: 'tooltip-container'
20-
})}
21-
>
22-
{!hideArrow && (
23-
<span
24-
{...getArrowProps({
25-
ref: arrowRef,
26-
'data-placement': placement,
27-
className: 'tooltip-arrow'
28-
})}
29-
/>
30-
)}
31-
{tooltip}
32-
</span>
33-
)}
34-
>
35-
{({ getTriggerProps, triggerRef }) => (
36-
<span
37-
{...getTriggerProps({
38-
ref: triggerRef
39-
})}
40-
>
41-
{children}
42-
</span>
43-
)}
44-
</TooltipTrigger>
45-
)
46-
);
5+
const BasicTooltipTrigger = memo(({tooltip, children, hideArrow, ...props}) => (
6+
<TooltipTrigger
7+
{...props}
8+
tooltip={({
9+
getTooltipProps,
10+
getArrowProps,
11+
tooltipRef,
12+
arrowRef,
13+
placement
14+
}) => (
15+
<span
16+
{...getTooltipProps({
17+
ref: tooltipRef,
18+
className: 'tooltip-container'
19+
})}
20+
>
21+
{!hideArrow && (
22+
<span
23+
{...getArrowProps({
24+
ref: arrowRef,
25+
'data-placement': placement,
26+
className: 'tooltip-arrow'
27+
})}
28+
/>
29+
)}
30+
{tooltip}
31+
</span>
32+
)}
33+
>
34+
{({getTriggerProps, triggerRef}) => (
35+
<span
36+
{...getTriggerProps({
37+
ref: triggerRef
38+
})}
39+
>
40+
{children}
41+
</span>
42+
)}
43+
</TooltipTrigger>
44+
));
4745

4846
export default BasicTooltipTrigger;

docs/examples/EscapeToClose.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ export default class EscapeToClose extends React.Component {
77
componentWillUnmount() {
88
document.removeEventListener('keydown', this._handleKeyDown);
99
}
10-
_handleKeyDown = ({ key }) => {
10+
_handleKeyDown = ({key}) => {
1111
if (key === 'Escape') {
1212
this.props.closeTooltip();
1313
}

docs/examples/utils.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@ export class StateContainer extends React.Component {
55
on: false
66
};
77

8-
set = on => this.setState({ on });
8+
set = on => this.setState({on});
99

10-
toggle = () => this.setState(prevState => ({ on: !prevState.on }));
10+
toggle = () => this.setState(prevState => ({on: !prevState.on}));
1111

1212
render() {
1313
return this.props.children({

rollup.config.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import babel from 'rollup-plugin-babel';
22
import resolve from 'rollup-plugin-node-resolve';
33
import commonjs from 'rollup-plugin-commonjs';
4-
import { terser } from 'rollup-plugin-terser';
5-
import { sizeSnapshot } from 'rollup-plugin-size-snapshot';
4+
import {terser} from 'rollup-plugin-terser';
5+
import {sizeSnapshot} from 'rollup-plugin-size-snapshot';
66
import pkg from './package.json';
77

88
const input = 'src/index.js';
@@ -13,10 +13,10 @@ const globals = {
1313
'react-popper': 'ReactPopper'
1414
};
1515
const external = id => !id.startsWith('.') && !id.startsWith('/');
16-
const getBabelOptions = ({ useESModules = true } = {}) => ({
16+
const getBabelOptions = ({useESModules = true} = {}) => ({
1717
exclude: 'node_modules/**',
1818
runtimeHelpers: true,
19-
plugins: [['@babel/plugin-transform-runtime', { useESModules }]]
19+
plugins: [['@babel/plugin-transform-runtime', {useESModules}]]
2020
});
2121

2222
export default [
@@ -51,7 +51,7 @@ export default [
5151
interop: false
5252
},
5353
external,
54-
plugins: [babel(getBabelOptions({ useESModules: false })), sizeSnapshot()]
54+
plugins: [babel(getBabelOptions({useESModules: false})), sizeSnapshot()]
5555
},
5656
{
5757
input,

src/Tooltip.js

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
/* eslint-disable react/no-find-dom-node */
2-
import React, { Component } from 'react';
3-
import { findDOMNode } from 'react-dom';
2+
import React, {Component} from 'react';
3+
import {findDOMNode} from 'react-dom';
44
import T from 'prop-types';
5-
import { TooltipContext } from './utils';
6-
import { callAll } from './utils';
5+
import {TooltipContext} from './utils';
6+
import {callAll} from './utils';
77

88
const MUTATION_OBSERVER_CONFIG = {
99
childList: true,
@@ -75,7 +75,7 @@ export default class Tooltip extends Component {
7575
document.removeEventListener('contextmenu', this._handleOutsideRightClick);
7676

7777
componentDidMount() {
78-
const { trigger } = this.props;
78+
const {trigger} = this.props;
7979
const observer = (this.observer = new MutationObserver(() => {
8080
this.props.scheduleUpdate();
8181
}));
@@ -95,7 +95,7 @@ export default class Tooltip extends Component {
9595
}
9696

9797
componentWillUnmount() {
98-
const { trigger } = this.props;
98+
const {trigger} = this.props;
9999
this.observer.disconnect();
100100

101101
if (trigger === 'click' || trigger === 'right-click') {
@@ -123,15 +123,15 @@ export default class Tooltip extends Component {
123123

124124
getArrowProps = (props = {}) => ({
125125
...props,
126-
style: { ...props.style, ...this.props.arrowProps.style }
126+
style: {...props.style, ...this.props.arrowProps.style}
127127
});
128128

129129
getTooltipProps = (props = {}) => {
130130
const isHoverTriggered = this.props.trigger === 'hover';
131131

132132
return {
133133
...props,
134-
style: { ...props.style, ...this.props.style },
134+
style: {...props.style, ...this.props.style},
135135
onMouseEnter: callAll(
136136
isHoverTriggered && this.props.clearScheduled,
137137
props.onMouseEnter
@@ -144,7 +144,7 @@ export default class Tooltip extends Component {
144144
};
145145

146146
render() {
147-
const { arrowProps, placement, tooltip, innerRef } = this.props;
147+
const {arrowProps, placement, tooltip, innerRef} = this.props;
148148

149149
return (
150150
<TooltipContext.Provider

0 commit comments

Comments
 (0)