javascript/ionic-framework
1
0
Fork 0
mirror of https://github.com/ionic-team/ionic-framework.git synced 2025-08-18 11:17:19 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(react): onIonTabsWillChange and onIonTabsDidChange event handlers are now properly bound to IonTabs (#22233)

Browse Source 
Co-authored-by: Hansu Kim <khsily@users.noreply.github.com>
This commit is contained in:
Ely Lucas
2021-02-11 12:50:07 -07:00
committed by GitHub
parent c05476b88e
commit b064fdebef
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
packages/react/src/components/navigation/IonTabs.tsx
View File

@ -85,6 +85,7 @@ export class IonTabs extends React.Component<Props> {
render() {
let outlet: React.ReactElement<{}> | undefined;
let tabBar: React.ReactElement | undefined;
const { className, onIonTabsDidChange, onIonTabsWillChange, ...props } = this.props;
const children =
typeof this.props.children === 'function'
@ -101,7 +102,7 @@ export class IonTabs extends React.Component<Props> {
outlet = child.props.children[0];
}
if (child.type === IonTabBar || child.type.isTabBar) {
const { onIonTabsDidChange, onIonTabsWillChange } = this.props;
tabBar = React.cloneElement(child, {
onIonTabsDidChange,
onIonTabsWillChange,
@ -111,7 +112,7 @@ export class IonTabs extends React.Component<Props> {
child.type === Fragment &&
(child.props.children[1].type === IonTabBar || child.props.children[1].type.isTabBar)
) {
const { onIonTabsDidChange, onIonTabsWillChange } = this.props;
tabBar = React.cloneElement(child.props.children[1], {
onIonTabsDidChange,
onIonTabsWillChange,
@ -127,8 +128,6 @@ export class IonTabs extends React.Component<Props> {
throw new Error('IonTabs needs a IonTabBar');
}
const { className, ...props } = this.props;
return (
<IonTabsContext.Provider value={this.ionTabContextState}>
{this.context.hasIonicRouter() ? (