Skip to content

Commit 2d784db

Browse files
committed
fix crashes in landscape mode
1 parent 554a659 commit 2d784db

3 files changed

Lines changed: 18 additions & 46 deletions

File tree

app/src/main/java/net/i2p/android/i2ptunnel/TunnelsContainer.java

Lines changed: 13 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -128,51 +128,14 @@ public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle sa
128128
@Override
129129
public void onActivityCreated(Bundle savedInstanceState) {
130130
super.onActivityCreated(savedInstanceState);
131-
// Add null check for safety
132-
if (mPageIndicator == null) {
133-
return;
134-
}
135-
131+
132+
// Initialize ViewPager and adapter
136133
mFragPagerAdapter = new TunnelsPagerAdapter(getChildFragmentManager());
137134
mViewPager.setAdapter(mFragPagerAdapter);
138135

139-
// Replace old indicator setup with MagicIndicator initialization
140-
CommonNavigator commonNavigator = new CommonNavigator(getContext());
141-
commonNavigator.setAdapter(new CommonNavigatorAdapter() {
142-
@Override
143-
public int getCount() {
144-
return mFragPagerAdapter.getCount();
145-
}
146-
147-
@Override
148-
public IPagerTitleView getTitleView(Context context, final int index) {
149-
SimplePagerTitleView simplePagerTitleView = new ColorTransitionPagerTitleView(context);
150-
simplePagerTitleView.setText(mFragPagerAdapter.getPageTitle(index));
151-
simplePagerTitleView.setNormalColor(ContextCompat.getColor(context,
152-
R.color.primary_text_disabled_material_dark));
153-
simplePagerTitleView.setSelectedColor(ContextCompat.getColor(context,
154-
R.color.primary_text_default_material_dark));
155-
simplePagerTitleView.setOnClickListener(new View.OnClickListener() {
156-
@Override
157-
public void onClick(View v) {
158-
mViewPager.setCurrentItem(index);
159-
}
160-
});
161-
return simplePagerTitleView;
162-
}
163-
164-
@Override
165-
public IPagerIndicator getIndicator(Context context) {
166-
LinePagerIndicator indicator = new LinePagerIndicator(context);
167-
indicator.setColors(ContextCompat.getColor(context, R.color.primary));
168-
return indicator;
169-
}
170-
});
171-
172-
mPageIndicator.setNavigator(commonNavigator);
173-
ViewPagerHelper.bind(mPageIndicator, mViewPager);
174-
//mPageIndicator.setViewPager(mViewPager);
136+
setupMagicIndicator();
175137

138+
// Setup New Tunnel button
176139
mNewTunnel.setOnClickListener(new View.OnClickListener() {
177140
@Override
178141
public void onClick(View view) {
@@ -378,6 +341,7 @@ private void setupMagicIndicator() {
378341
}
379342

380343
CommonNavigator commonNavigator = new CommonNavigator(getContext());
344+
commonNavigator.setAdjustMode(true); // Add this line for better spacing
381345
commonNavigator.setAdapter(new CommonNavigatorAdapter() {
382346
@Override
383347
public int getCount() {
@@ -388,6 +352,7 @@ public int getCount() {
388352
public IPagerTitleView getTitleView(Context context, final int index) {
389353
SimplePagerTitleView simplePagerTitleView = new ColorTransitionPagerTitleView(context);
390354
simplePagerTitleView.setText(mFragPagerAdapter.getPageTitle(index));
355+
simplePagerTitleView.setTextSize(16); // Add this line to increase text size
391356
simplePagerTitleView.setNormalColor(ContextCompat.getColor(context,
392357
R.color.primary_text_disabled_material_dark));
393358
simplePagerTitleView.setSelectedColor(ContextCompat.getColor(context,
@@ -404,12 +369,19 @@ public void onClick(View v) {
404369
@Override
405370
public IPagerIndicator getIndicator(Context context) {
406371
LinePagerIndicator indicator = new LinePagerIndicator(context);
372+
indicator.setMode(LinePagerIndicator.MODE_WRAP_CONTENT);
407373
indicator.setColors(ContextCompat.getColor(context, R.color.primary));
374+
indicator.setLineHeight(dpToPx(context, 3));
408375
return indicator;
409376
}
410377
});
411378

412379
mPageIndicator.setNavigator(commonNavigator);
413380
ViewPagerHelper.bind(mPageIndicator, mViewPager);
414381
}
382+
383+
private int dpToPx(Context context, int dp) {
384+
float density = context.getResources().getDisplayMetrics().density;
385+
return Math.round(dp * density);
386+
}
415387
}

app/src/main/res/layout-land/activity_viewpager.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
android:layout_height="match_parent"
66
android:orientation="vertical">
77

8-
<android.support.v7.widget.Toolbar
8+
<androidx.appcompat.widget.Toolbar
99
android:id="@+id/main_toolbar"
1010
android:layout_width="match_parent"
1111
android:layout_height="?attr/actionBarSize"
@@ -18,7 +18,7 @@
1818
android:id="@+id/sliding_tabs"
1919
android:layout_width="wrap_content"
2020
android:layout_height="wrap_content" />
21-
</android.support.v7.widget.Toolbar>
21+
</androidx.appcompat.widget.Toolbar>
2222

2323
<net.i2p.android.widget.DrawShadowFrameLayout
2424
android:layout_width="match_parent"
@@ -27,7 +27,7 @@
2727
android:clipChildren="false"
2828
app:shadowDrawable="@drawable/header_shadow">
2929

30-
<android.support.v4.view.CustomViewPager
30+
<androidx.viewpager.widget.CustomViewPager
3131
android:id="@+id/pager"
3232
android:layout_width="match_parent"
3333
android:layout_height="match_parent" />

app/src/main/res/layout-land/fragment_main.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
</LinearLayout>
7373
</LinearLayout>
7474

75-
<android.support.v7.widget.Toolbar
75+
<androidx.appcompat.widget.Toolbar
7676
android:id="@+id/console_bottom_toolbar"
7777
android:layout_width="match_parent"
7878
android:layout_height="56dp"
@@ -130,7 +130,7 @@
130130
tools:text="0.0 KBps / 0.0 KB" />
131131
</LinearLayout>
132132
</LinearLayout>
133-
</android.support.v7.widget.Toolbar>
133+
</androidx.appcompat.widget.Toolbar>
134134
</RelativeLayout>
135135

136136
<ScrollView

0 commit comments

Comments
 (0)