Skip to content

Commit 2c19ab4

Browse files
committed
fix(super-tabs): onTabChange firing twice if page was not changed
1 parent 36a8c66 commit 2c19ab4

File tree

2 files changed

+1
-8
lines changed

2 files changed

+1
-8
lines changed

src/components/super-tabs-container.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,6 @@ export class SuperTabsContainer implements AfterViewInit, OnDestroy {
200200
* @param index {number} tab index
201201
*/
202202
private setSelectedTab(index: number) {
203-
console.log('Setting selected tab to: ' + index + ', from: ' + this.selectedTabIndex);
204203
this.tabSelect.emit({ index, changed: index !== this.selectedTabIndex });
205204
this.selectedTabIndex = index;
206205
}

src/components/super-tabs.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -488,20 +488,16 @@ export class SuperTabs implements OnInit, AfterContentInit, AfterViewInit, OnDes
488488
onTabChange(index: number) {
489489
index = Number(index);
490490
if (index === this.selectedTabIndex) {
491-
console.info('[a] ', index, this.selectedTabIndex);
492-
// console.trace();
493491
this.tabSelect.emit({
494492
index,
495493
id: this._tabs[index].tabId,
496494
changed: false
497495
});
496+
return;
498497
}
499498

500-
console.info('[b] ', index, this.selectedTabIndex);
501499

502500
if (index <= this._tabs.length) {
503-
console.info('[c] ', index, this.selectedTabIndex);
504-
// console.trace();
505501

506502
const currentTab: SuperTab = this.getActiveTab();
507503
let activeView: ViewController = currentTab.getActive();
@@ -533,15 +529,13 @@ export class SuperTabs implements OnInit, AfterContentInit, AfterViewInit, OnDes
533529
}
534530

535531
onToolbarTabSelect(index: number) {
536-
console.log('toolbar tab selected', index);
537532
if (index !== this.selectedTabIndex) {
538533
this.tabsContainer.slideTo(index);
539534
}
540535
this.onTabChange(index);
541536
}
542537

543538
onContainerTabSelect(ev: { index: number; changed: boolean }) {
544-
console.log('Container tab select fired! ', ev);
545539
if (ev.changed) {
546540
this.onTabChange(ev.index);
547541
}

0 commit comments

Comments
 (0)