8000 Merge pull request #7561 from NativeScript/myankov/merge-release · NativeScript/NativeScript@49ab364 · GitHub
[go: up one dir, main page]

Skip to content

Commit 49ab364

Browse files
authored
Merge pull request #7561 from NativeScript/myankov/merge-release
chore: merge release to master
2 parents 46db840 + 8e057c7 commit 49ab364

File tree

5 files changed

+27
-11
lines changed

5 files changed

+27
-11
lines changed

tns-core-modules-widgets/android/widgets/src/main/java/org/nativescript/widgets/BottomNavigationBar.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -216,8 +216,9 @@ private void setupItem(LinearLayout ll, TextView textView,ImageView imgView, Tab
216216
lp.weight = 1;
217217
}
218218

219-
public void onTap(int position) {
219+
public boolean onTap(int position) {
220220
// to be overridden in JS
221+
return true;
221222
}
222223

223224
public void onSelectedPositionChange(int position, int prevPosition) {
@@ -269,8 +270,9 @@ private class TabClickListener implements OnClickListener {
269270
public void onClick(View v) {
270271
for (int i = 0; i < mTabStrip.getChildCount(); i++) {
271272
if (v == mTabStrip.getChildAt(i)) {
272-
onTap(i);
273-
setSelectedPosition(i);
273+
if (onTap(i)) {
274+
setSelectedPosition(i);
275+
}
274276
return;
275277
}
276278
}

tns-core-modules-widgets/android/widgets/src/main/java/org/nativescript/widgets/TabLayout.java

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -301,8 +301,9 @@ private void setupItem(LinearLayout ll, TextView textView,ImageView imgView, Tab
301301
}
302302
}
303303

304-
public void onTap(int position) {
304+
public boolean onTap(int position) {
305305
// to be overridden in JS
306+
return true;
306307
}
307308

308309
public void onSelectedPositionChange(int position, int prevPosition) {
@@ -429,8 +430,9 @@ private class TabClickListener implements OnClickListener {
429430
public void onClick(View v) {
430431
for (int i = 0; i < mTabStrip.getChildCount(); i++) {
431432
if (v == mTabStrip.getChildAt(i)) {
432-
onTap(i);
433-
mViewPager.setCurrentItem(i);
433+
if (onTap(i)) {
434+
mViewPager.setCurrentItem(i);
435+
}
434436
return;
435437
}
436438
}

tns-core-modules/ui/bottom-navigation/bottom-navigation.android.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,17 +118,23 @@ function initializeNativeClasses() {
118118
owner.selectedIndex = position;
119119
}
120120

121-
public onTap(position: number): void {
121+
public onTap(position: number): boolean {
122122
const owner = this.owner;
123123
if (!owner) {
124-
return;
124+
return false;
125125
}
126126

127127
const tabStripItems = owner.tabStrip && owner.tabStrip.items;
128128

129129
if (position >= 0 && tabStripItems[position]) {
130130
tabStripItems[position]._emit(TabStripItem.tapEvent);
131131
}
132+
133+
if (!owner.items[position]) {
134+
return false;
135+
}
136+
137+
return true;
132138
}
133139
}
134140

tns-core-modules/ui/tabs/tabs.android.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -259,17 +259,23 @@ function initializeNativeClasses() {
259259
}
260260
}
261261

262-
public onTap(position: number): void {
262+
public onTap(position: number): boolean {
263263
const owner = this.owner;
264264
if (!owner) {
265-
return;
265+
return false;
266266
}
267267

268268
const tabStripItems = owner.tabStrip && owner.tabStrip.items;
269269

270270
if (position >= 0 && tabStripItems[position]) {
271271
tabStripItems[position]._emit(TabStripItem.tapEvent);
272272
}
273+
274+
if (!owner.items[position]) {
275+
return false;
276+
}
277+
278+
return true;
273279
}
274280
}
275281

tns-platform-declarations/android/org.nativescript.widgets.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -410,7 +410,7 @@
410410
setTabTextFontSize(fontSize: number): void;
411411
getTabTextFontSize(): number;
412412

413-
onTap(position: number): void;
413+
onTap(position: number): boolean;
414414
onSelectedPositionChange(position: number, prevPosition: number): void ;
415415
setSelectedPosition(position: number): void;
416416
setItems(items: Array<TabItemSpec>): void;

0 commit comments

Comments
 (0)
0