Skip to content

Commit 17a8788

Browse files
luisguzman-adfaluisguzman-adfa
committed
ADFA-4886: Material 3 nav bar for the WebView chrome (icons, theme, a11y, i18n)
Replace the PortalActivity nav bar's emoji/glyph buttons with tinted Material 3 vector ImageButtons (back, fullscreen, reload/stop, exit, forward, plus the show/hide handles) and move PortalActivity onto Theme.K2Go (Material3 DayNight + app typography) so the chrome follows the system light/dark. Surfaces and tints come from theme attributes; Exit keeps its universal red. Add contentDescription to the icon-only buttons and move the WebView error-page strings into resources. Strings are translatable=false pending the l10n pass.
1 parent c16937e commit 17a8788

11 files changed

Lines changed: 140 additions & 63 deletions

File tree

‎controller/app/src/main/AndroidManifest.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@
7171
android:exported="false"
7272
android:configChanges="orientation|screenSize"
7373
android:hardwareAccelerated="true"
74-
android:theme="@style/Theme.AppCompat.DayNight.NoActionBar"/>
74+
android:theme="@style/Theme.K2Go"/> <!-- ADFA-4886: Material 3 DayNight + app typography for the WebView chrome -->
7575

7676
<activity
7777
android:name="com.journeyapps.barcodescanner.CaptureActivity"

‎controller/app/src/main/java/org/iiab/controller/PortalActivity.java‎

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
import android.graphics.Bitmap;
2323
import android.view.View;
24-
import android.widget.Button;
24+
import android.widget.ImageButton;
2525
import android.widget.LinearLayout;
2626
import android.content.Intent;
2727
import android.os.Handler;
@@ -95,14 +95,14 @@ protected void onCreate(Bundle savedInstanceState) {
9595
webView.setGestureLogging(BuildConfig.DEBUG);
9696

9797
LinearLayout bottomNav = findViewById(R.id.bottomNav);
98-
Button btnHandle = findViewById(R.id.btnHandle); // The new handle
99-
Button btnHideNav = findViewById(R.id.btnHideNav); // Button to close
98+
ImageButton btnHandle = findViewById(R.id.btnHandle); // The new handle
99+
ImageButton btnHideNav = findViewById(R.id.btnHideNav); // Button to close
100100

101-
Button btnBack = findViewById(R.id.btnBack);
102-
Button btnHome = findViewById(R.id.btnHome);
103-
Button btnReload = findViewById(R.id.btnReload);
104-
Button btnExit = findViewById(R.id.btnExit);
105-
Button btnForward = findViewById(R.id.btnForward);
101+
ImageButton btnBack = findViewById(R.id.btnBack);
102+
ImageButton btnHome = findViewById(R.id.btnHome);
103+
ImageButton btnReload = findViewById(R.id.btnReload);
104+
ImageButton btnExit = findViewById(R.id.btnExit);
105+
ImageButton btnForward = findViewById(R.id.btnForward);
106106

107107
// --- NAV BAR (auto-hide after inactivity) ---
108108
// Visible on entry; hides after AUTO_HIDE_MS with no interaction, measured from the LAST
@@ -206,14 +206,16 @@ public boolean shouldOverrideUrlLoading(WebView view, android.webkit.WebResource
206206
public void onPageStarted(WebView view, String url, Bitmap favicon) {
207207
super.onPageStarted(view, url, favicon);
208208
vm.setLoading(true);
209-
btnReload.setText("✕"); // Change to Stop
209+
btnReload.setImageResource(R.drawable.ic_stop); // Change to Stop
210+
btnReload.setContentDescription(getString(R.string.k2go_nav_stop));
210211
}
211212

212213
@Override
213214
public void onPageFinished(WebView view, String url) {
214215
super.onPageFinished(view, url);
215216
vm.setLoading(false);
216-
btnReload.setText("↻"); // Back to Reload
217+
btnReload.setImageResource(R.drawable.ic_refresh); // Back to Reload
218+
btnReload.setContentDescription(getString(R.string.k2go_nav_reload));
217219
view.getSettings().setCacheMode(android.webkit.WebSettings.LOAD_DEFAULT);
218220

219221
// Touch diagnostics + best-effort MapLibre pitch enablement.
@@ -228,13 +230,15 @@ public void onReceivedError(WebView view, android.webkit.WebResourceRequest requ
228230
super.onReceivedError(view, request, error);
229231
if (request.isForMainFrame()) {
230232
String customErrorHtml = "<html><body style='background-color:#1A1A1A;color:#FFFFFF;text-align:center;padding-top:50px;font-family:sans-serif;'>"
231-
+ "<h2>⚠️ Connection Failed</h2>"
232-
+ "<p>Unable to reach the secure environment.</p>"
233-
+ "<p style='color:#888;font-size:12px;'>Error: " + error.getDescription() + "</p>"
233+
+ "<h2>⚠️ " + getString(R.string.k2go_portal_error_title) + "</h2>"
234+
+ "<p>" + getString(R.string.k2go_portal_error_body) + "</p>"
235+
+ "<p style='color:#888;font-size:12px;'>"
236+
+ getString(R.string.k2go_portal_error_detail, error.getDescription()) + "</p>"
234237
+ "</body></html>";
235238
view.loadData(customErrorHtml, "text/html", "UTF-8");
236239
vm.setLoading(false);
237-
btnReload.setText("↻");
240+
btnReload.setImageResource(R.drawable.ic_refresh);
241+
btnReload.setContentDescription(getString(R.string.k2go_nav_reload));
238242
}
239243
}
240244
});

‎controller/app/src/main/res/drawable/ic_arrow_right.xml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
2+
android:width="24dp" android:height="24dp"
3+
android:viewportWidth="24" android:viewportHeight="24"
4+
android:tint="?attr/colorControlNormal">
5+
<path android:fillColor="#FF000000"
6+
android:pathData="M12,4l-1.41,1.41L16.17,11H4v2h12.17l-5.58,5.59L12,20l8,-8z"/>
7+
</vector>

‎controller/app/src/main/res/drawable/ic_expand_less.xml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
2+
android:width="24dp" android:height="24dp"
3+
android:viewportWidth="24" android:viewportHeight="24"
4+
android:tint="?attr/colorControlNormal">
5+
<path android:fillColor="#FF000000"
6+
android:pathData="M12,8l-6,6 1.41,1.41L12,10.83l4.59,4.58L18,14z"/>
7+
</vector>

‎controller/app/src/main/res/drawable/ic_expand_more.xml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
2+
android:width="24dp" android:height="24dp"
3+
android:viewportWidth="24" android:viewportHeight="24"
4+
android:tint="?attr/colorControlNormal">
5+
<path android:fillColor="#FF000000"
6+
android:pathData="M16.59,8.59L12,13.17 7.41,8.59 6,10l6,6 6,-6z"/>
7+
</vector>

‎controller/app/src/main/res/drawable/ic_fullscreen.xml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
2+
android:width="24dp" android:height="24dp"
3+
android:viewportWidth="24" android:viewportHeight="24"
4+
android:tint="?attr/colorControlNormal">
5+
<path android:fillColor="#FF000000"
6+
android:pathData="M7,14H5v5h5v-2H7v-3zM5,10h2V7h3V5H5v5zM17,17h-3v2h5v-5h-2v3zM14,5v2h3v3h2V5h-5z"/>
7+
</vector>

‎controller/app/src/main/res/drawable/ic_refresh.xml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
2+
android:width="24dp" android:height="24dp"
3+
android:viewportWidth="24" android:viewportHeight="24"
4+
android:tint="?attr/colorControlNormal">
5+
<path android:fillColor="#FF000000"
6+
android:pathData="M17.65,6.35C16.2,4.9 14.21,4 12,4c-4.42,0 -7.99,3.58 -7.99,8s3.57,8 7.99,8c3.73,0 6.84,-2.55 7.73,-6h-2.08c-0.82,2.33 -3.04,4 -5.65,4 -3.31,0 -6,-2.69 -6,-6s2.69,-6 6,-6c1.66,0 3.14,0.69 4.22,1.78L13,11h7V4l-2.35,2.35z"/>
7+
</vector>

‎controller/app/src/main/res/drawable/ic_stop.xml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
<vector xmlns:android="http://schemas.android.com/apk/res/android"
2+
android:width="24dp" android:height="24dp"
3+
android:viewportWidth="24" android:viewportHeight="24"
4+
android:tint="?attr/colorControlNormal">
5+
<path android:fillColor="#FF000000"
6+
android:pathData="M6,6h12v12H6z"/>
7+
</vector>

‎controller/app/src/main/res/layout/activity_portal.xml‎

Lines changed: 61 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,9 @@
11
<?xml version="1.0" encoding="utf-8"?>
22
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
3+
xmlns:app="http://schemas.android.com/apk/res-auto"
34
android:layout_width="match_parent"
45
android:layout_height="match_parent"
5-
android:background="?android:attr/colorBackground">
6+
android:background="?attr/colorSurface">
67

78
<!-- WebView (Web page content handles its own background,
89
but you can force a dark theme in Java using WebSettings.setForceDark) -->
@@ -11,98 +12,110 @@
1112
android:layout_width="match_parent"
1213
android:layout_height="match_parent" />
1314

14-
<Button
15+
<!-- ADFA-4886: nav chrome uses Material 3 vector icons + theme attributes (system day/night)
16+
instead of emoji/glyph text buttons. Icon-only buttons carry a contentDescription. -->
17+
<ImageButton
1518
android:id="@+id/btnHandle"
1619
android:layout_width="60dp"
1720
android:layout_height="32dp"
1821
android:layout_alignParentBottom="true"
1922
android:layout_centerHorizontal="true"
2023
android:alpha="0.8"
21-
android:background="?android:attr/colorButtonNormal"
22-
android:padding="0dp"
23-
android:stateListAnimator="@null"
24-
android:text="⌃"
25-
android:textColor="?android:attr/textColorPrimary"
26-
android:textSize="18sp" />
24+
android:background="?attr/colorSurfaceContainerHighest"
25+
android:contentDescription="@string/k2go_nav_show_bar"
26+
android:padding="4dp"
27+
android:scaleType="fitCenter"
28+
android:tint="?attr/colorOnSurfaceVariant"
29+
app:srcCompat="@drawable/ic_expand_less" />
2730

2831
<LinearLayout
2932
android:id="@+id/bottomNav"
3033
android:layout_width="match_parent"
3134
android:layout_height="wrap_content"
3235
android:layout_alignParentBottom="true"
33-
android:background="?android:attr/colorBackgroundFloating"
36+
android:background="?attr/colorSurfaceContainer"
3437
android:orientation="vertical"
3538
android:visibility="invisible"
3639
android:elevation="8dp">
3740

38-
<Button
41+
<ImageButton
3942
android:id="@+id/btnHideNav"
4043
android:layout_width="match_parent"
4144
android:layout_height="30dp"
42-
android:background="?android:attr/selectableItemBackground"
43-
android:text="⌄"
44-
android:textSize="20sp"
45-
android:textColor="?android:attr/textColorSecondary"
46-
android:stateListAnimator="@null" />
45+
android:background="?attr/selectableItemBackground"
46+
android:contentDescription="@string/k2go_nav_hide_bar"
47+
android:padding="4dp"
48+
android:scaleType="fitCenter"
49+
android:tint="?attr/colorOnSurfaceVariant"
50+
app:srcCompat="@drawable/ic_expand_more" />
4751

4852
<LinearLayout
4953
android:layout_width="match_parent"
5054
android:layout_height="wrap_content"
5155
android:orientation="horizontal"
5256
android:paddingBottom="8dp">
5357

54-
<Button
58+
<ImageButton
5559
android:id="@+id/btnBack"
5660
android:layout_width="0dp"
57-
android:layout_height="wrap_content"
61+
android:layout_height="48dp"
5862
android:layout_weight="1"
59-
android:text="◀"
60-
android:textSize="20sp"
61-
style="?android:attr/buttonBarButtonStyle"
62-
android:textColor="?android:attr/textColorPrimary" />
63+
android:background="?attr/selectableItemBackgroundBorderless"
64+
android:contentDescription="@string/k2go_nav_back"
65+
android:padding="12dp"
66+
android:scaleType="fitCenter"
67+
android:tint="?attr/colorOnSurface"
68+
app:srcCompat="@drawable/ic_arrow_left" />
6369

64-
<!-- ADFA-4887: repurposed as a Fullscreen toggle (no in-WebView home page anymore).
65-
Material 3 icon + contentDescription land in ADFA-4886. -->
66-
<Button
70+
<!-- ADFA-4887: repurposed as a Fullscreen toggle (no in-WebView home page anymore). -->
71+
<ImageButton
6772
android:id="@+id/btnHome"
6873
android:layout_width="0dp"
69-
android:layout_height="wrap_content"
74+
android:layout_height="48dp"
7075
android:layout_weight="1"
71-
android:text="⛶"
72-
android:textSize="20sp"
73-
style="?android:attr/buttonBarButtonStyle"
74-
android:textColor="?android:attr/textColorPrimary" />
76+
android:background="?attr/selectableItemBackgroundBorderless"
77+
android:contentDescription="@string/k2go_nav_fullscreen"
78+
android:padding="12dp"
79+
android:scaleType="fitCenter"
80+
android:tint="?attr/colorOnSurface"
81+
app:srcCompat="@drawable/ic_fullscreen" />
7582

76-
<Button
83+
<ImageButton
7784
android:id="@+id/btnReload"
7885
android:layout_width="0dp"
79-
android:layout_height="wrap_content"
86+
android:layout_height="48dp"
8087
android:layout_weight="1"
81-
android:text="↻"
82-
android:textSize="20sp"
83-
style="?android:attr/buttonBarButtonStyle"
84-
android:textColor="?android:attr/textColorPrimary" />
88+
android:background="?attr/selectableItemBackgroundBorderless"
89+
android:contentDescription="@string/k2go_nav_reload"
90+
android:padding="12dp"
91+
android:scaleType="fitCenter"
92+
android:tint="?attr/colorOnSurface"
93+
app:srcCompat="@drawable/ic_refresh" />
8594

86-
<Button
95+
<ImageButton
8796
android:id="@+id/btnExit"
8897
android:layout_width="0dp"
89-
android:layout_height="wrap_content"
98+
android:layout_height="48dp"
9099
android:layout_weight="1"
91-
android:text="✖"
92-
android:textSize="20sp"
93-
style="?android:attr/buttonBarButtonStyle"
94-
android:textColor="#D32F2F" /> <!-- Red is universal alert color -->
100+
android:background="?attr/selectableItemBackgroundBorderless"
101+
android:contentDescription="@string/k2go_nav_exit"
102+
android:padding="12dp"
103+
android:scaleType="fitCenter"
104+
android:tint="@color/k2go_nav_exit"
105+
app:srcCompat="@drawable/ic_close_24" /> <!-- Red is the universal alert color -->
95106

96-
<Button
107+
<ImageButton
97108
android:id="@+id/btnForward"
98109
android:layout_width="0dp"
99-
android:layout_height="wrap_content"
110+
android:layout_height="48dp"
100111
android:layout_weight="1"
101-
android:text="▶"
102-
android:textSize="20sp"
103-
style="?android:attr/buttonBarButtonStyle"
104-
android:textColor="?android:attr/textColorPrimary" />
112+
android:background="?attr/selectableItemBackgroundBorderless"
113+
android:contentDescription="@string/k2go_nav_forward"
114+
android:padding="12dp"
115+
android:scaleType="fitCenter"
116+
android:tint="?attr/colorOnSurface"
117+
app:srcCompat="@drawable/ic_arrow_right" />
105118
</LinearLayout>
106119

107120
</LinearLayout>
108-
</RelativeLayout>
121+
</RelativeLayout>

‎controller/app/src/main/res/values/colors_k2go.xml‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,4 +36,8 @@
3636
<color name="k2go_cover_6">#4A5568</color> <!-- slate -->
3737
<color name="k2go_band_library">#2E9E5B</color> <!-- green band: already in your books -->
3838
<color name="k2go_band_selected">#4A504E</color> <!-- gray band: selected to add -->
39+
40+
<!-- ADFA-4886: WebView nav "Exit" tint. Red is the universal alert/exit color; kept fixed in
41+
both light and dark (no values-night override) so the exit affordance always reads the same. -->
42+
<color name="k2go_nav_exit">#D32F2F</color>
3943
</resources>

0 commit comments

Comments
 (0)