Merge pull request #7561 from NativeScript/myankov/merge-release

chore: merge release to master
This commit is contained in:
Martin Yankov
2019-07-22 11:48:22 +03:00
committed by GitHub
5 changed files with 27 additions and 11 deletions

View File

@ -216,8 +216,9 @@ public class BottomNavigationBar extends LinearLayout {
lp.weight = 1; lp.weight = 1;
} }
public void onTap(int position) { public boolean onTap(int position) {
// to be overridden in JS // to be overridden in JS
return true;
} }
public void onSelectedPositionChange(int position, int prevPosition) { public void onSelectedPositionChange(int position, int prevPosition) {
@ -269,8 +270,9 @@ public class BottomNavigationBar extends LinearLayout {
public void onClick(View v) { public void onClick(View v) {
for (int i = 0; i < mTabStrip.getChildCount(); i++) { for (int i = 0; i < mTabStrip.getChildCount(); i++) {
if (v == mTabStrip.getChildAt(i)) { if (v == mTabStrip.getChildAt(i)) {
onTap(i); if (onTap(i)) {
setSelectedPosition(i); setSelectedPosition(i);
}
return; return;
} }
} }

View File

@ -301,8 +301,9 @@ public class TabLayout extends HorizontalScrollView {
} }
} }
public void onTap(int position) { public boolean onTap(int position) {
// to be overridden in JS // to be overridden in JS
return true;
} }
public void onSelectedPositionChange(int position, int prevPosition) { public void onSelectedPositionChange(int position, int prevPosition) {
@ -429,8 +430,9 @@ public class TabLayout extends HorizontalScrollView {
public void onClick(View v) { public void onClick(View v) {
for (int i = 0; i < mTabStrip.getChildCount(); i++) { for (int i = 0; i < mTabStrip.getChildCount(); i++) {
if (v == mTabStrip.getChildAt(i)) { if (v == mTabStrip.getChildAt(i)) {
onTap(i); if (onTap(i)) {
mViewPager.setCurrentItem(i); mViewPager.setCurrentItem(i);
}
return; return;
} }
} }

View File

@ -118,10 +118,10 @@ function initializeNativeClasses() {
owner.selectedIndex = position; owner.selectedIndex = position;
} }
public onTap(position: number): void { public onTap(position: number): boolean {
const owner = this.owner; const owner = this.owner;
if (!owner) { if (!owner) {
return; return false;
} }
const tabStripItems = owner.tabStrip && owner.tabStrip.items; const tabStripItems = owner.tabStrip && owner.tabStrip.items;
@ -129,6 +129,12 @@ function initializeNativeClasses() {
if (position >= 0 && tabStripItems[position]) { if (position >= 0 && tabStripItems[position]) {
tabStripItems[position]._emit(TabStripItem.tapEvent); tabStripItems[position]._emit(TabStripItem.tapEvent);
} }
if (!owner.items[position]) {
return false;
}
return true;
} }
} }

View File

@ -259,10 +259,10 @@ function initializeNativeClasses() {
} }
} }
public onTap(position: number): void { public onTap(position: number): boolean {
const owner = this.owner; const owner = this.owner;
if (!owner) { if (!owner) {
return; return false;
} }
const tabStripItems = owner.tabStrip && owner.tabStrip.items; const tabStripItems = owner.tabStrip && owner.tabStrip.items;
@ -270,6 +270,12 @@ function initializeNativeClasses() {
if (position >= 0 && tabStripItems[position]) { if (position >= 0 && tabStripItems[position]) {
tabStripItems[position]._emit(TabStripItem.tapEvent); tabStripItems[position]._emit(TabStripItem.tapEvent);
} }
if (!owner.items[position]) {
return false;
}
return true;
} }
} }

View File

@ -410,7 +410,7 @@
setTabTextFontSize(fontSize: number): void; setTabTextFontSize(fontSize: number): void;
getTabTextFontSize(): number; getTabTextFontSize(): number;
onTap(position: number): void; onTap(position: number): boolean;
onSelectedPositionChange(position: number, prevPosition: number): void ; onSelectedPositionChange(position: number, prevPosition: number): void ;
setSelectedPosition(position: number): void; setSelectedPosition(position: number): void;
setItems(items: Array<TabItemSpec>): void; setItems(items: Array<TabItemSpec>): void;