K2GO-385 refactor(ui): status badge on its own line below the metadata tags (detail screens)

On-device review of PR3b: the dot+text status badge shared the metadata chip row
with the boxed metadata tags (e.g. "v1.2.12 [] . Up to date . REST API []"), and
the FlowLayout top-aligns its children, so the badge floated off the tags' centre
axis -- the ragged mix the design decision warns against. Give the status its own
zone, as the decision prescribes ("metadata + status cluster ... status on its own
line so the roles don't share a shape row").

- New k2go_moddet_status FlowLayout in the shared detail layout, directly below the
  metadata chips, GONE until a status is added.
- DashboardDetail: the live status badge moves out of the chip row into it (the chip
  row keeps version / REST API / System core).
- ModuleDetail: the lifecycle state badges move there too (via addStatus, which
  reveals the row); the metadata + trait tags (size / version / Runs offline / Demo)
  stay in the chip row. Drops the now-unused chipRow alias.

Fixes the "shared centre axis" finding from the PR3b review.

Author: luisguzman-adfa

controller/app/src/main/java/org/appdevforall/k2go/redesign/DashboardDetailFragment.java

@@ -103,13 +103,14 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
         ((TextView) root.findViewById(R.id.k2go_moddet_desc)).setText(R.string.k2go_dash_detail_desc);
 
         // Meta chips, in order: version (prepended live) | update status (live) | REST API | System core.
-        // ADFA-5026: the status chip starts at index 0 so that when the version chip prepends at 0 it
-        // lands right after the version. "Runs offline" is replaced by the update-status pill.
         chips = root.findViewById(R.id.k2go_moddet_chips);
-        // K2GO-385 (PR3): the live lifecycle state is a status badge (dot+text, semantic); REST API /
-        // System core are neutral metadata tags (colour there is noise -- the metadata colour rule).
+        // K2GO-385 (PR3): the live lifecycle state is a status badge (dot+text, semantic) on its OWN line
+        // below the metadata; version / REST API / System core are neutral metadata tags in the chip row
+        // (colour there is noise -- the metadata colour rule). The version chip prepends to the chip row.
+        ViewGroup statusRow = root.findViewById(R.id.k2go_moddet_status);
+        statusRow.setVisibility(View.VISIBLE);
         statusChip = K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_dash_chip_checking), R.color.k2go_muted);
-        chips.addView(statusChip);
+        statusRow.addView(statusChip);
         chips.addView(K2GoChip.create(requireContext(), getString(R.string.k2go_dash_chip_rest)));
         chips.addView(K2GoChip.create(requireContext(), getString(R.string.k2go_dash_chip_core)));
         fetchVersionChip();

controller/app/src/main/java/org/appdevforall/k2go/redesign/ModuleDetailFragment.java

@@ -99,7 +99,7 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
         // than start visible, because showing them and then taking them away is how a user ends
         // up tapping one in the gap.
         final Button installNowBtn = root.findViewById(R.id.k2go_moddet_install_now);
-        final ViewGroup chipRow = chips;
+        final ViewGroup statusRow = root.findViewById(R.id.k2go_moddet_status);
         installNowBtn.setVisibility(View.GONE);
 
         Button schedule = root.findViewById(R.id.k2go_moddet_schedule);
@@ -125,7 +125,7 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
                 if (!isAdded()) return;
                 if (verdict == org.appdevforall.k2go.system.domain.SystemVerdict.State.NO_SYSTEM
                         || verdict == org.appdevforall.k2go.system.domain.SystemVerdict.State.DAMAGED) {
-                    chipRow.addView(K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_state_no_system), R.color.k2go_amber_text));
+                    addStatus(statusRow, K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_state_no_system), R.color.k2go_amber_text));
                     installNowBtn.setText(R.string.k2go_home_recover);
                     installNowBtn.setOnClickListener(v -> SetupLibraryActivity.recover(requireContext()));
                     installNowBtn.setVisibility(View.VISIBLE);
@@ -136,11 +136,11 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
                         || verdict == org.appdevforall.k2go.system.domain.SystemVerdict.State.CLONE_SHARING) {
                     // ADFA-5312: a system op is in progress — the system is present but mid-setup and the
                     // server is down. Don't offer Install or Recover into it; just say it's busy.
-                    chipRow.addView(K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_home_installing), R.color.k2go_amber_text));
+                    addStatus(statusRow, K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_home_installing), R.color.k2go_amber_text));
                     return;
                 }
                 if (isInstalled) {
-                    chipRow.addView(K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_mod_phase_done), R.color.k2go_leaf));
+                    addStatus(statusRow, K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_mod_phase_done), R.color.k2go_leaf));
                     return;   // nothing to offer: a module cannot be uninstalled or reinstalled here
                 }
                 // ADFA-4898: this module's runrole failed in the last finished batch — the SAME per-module
@@ -153,7 +153,7 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
                 // Retry then bounces to the hub, which observes the queue live; this detail is a one-shot
                 // snapshot (no observer) and would otherwise sit on a stale "Couldn't install".
                 if (org.appdevforall.k2go.install.presentation.ModuleQueueRepository.get().current().didFail(c.key())) {
-                    chipRow.addView(K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_mod_phase_failed), R.color.k2go_clay));
+                    addStatus(statusRow, K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_mod_phase_failed), R.color.k2go_clay));
                     schedule.setText(R.string.k2go_home_retry);
                     // Shared, busy-gated retry (same action the live progress card fires). On a real start,
                     // land on the install index — the same destination as a normal install (openModuleIndex)
@@ -172,7 +172,7 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
                     return;
                 }
                 if (unknown) {
-                    chipRow.addView(K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_state_no_answer),
+                    addStatus(statusRow, K2GoStatusBadge.create(requireContext(), getString(R.string.k2go_state_no_answer),
                             R.color.k2go_amber_text));
                     return;   // no grounds to offer work either way
                 }
@@ -208,4 +208,11 @@ public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup c
         return root;
     }
 
+    /** K2GO-385 (PR3): a lifecycle status badge goes on its own row (k2go_moddet_status), which stays
+     *  GONE until one is added -- so a dot+text status never shares the boxed-metadata chip row. */
+    private static void addStatus(ViewGroup row, View badge) {
+        row.setVisibility(View.VISIBLE);
+        row.addView(badge);
+    }
+
 }

controller/app/src/main/res/layout/fragment_k2go_module_detail.xml

@@ -66,6 +66,16 @@
                 android:layout_height="wrap_content"
                 android:layout_marginTop="12dp" />
 
+            <!-- K2GO-385 (PR3): the lifecycle status sits on its OWN line below the metadata tags, so a
+                 dot+text status badge never shares a row (and a centre axis) with the boxed metadata tags.
+                 GONE until a status is added (some module states show none). -->
+            <org.appdevforall.k2go.redesign.FlowLayout
+                android:id="@+id/k2go_moddet_status"
+                android:layout_width="match_parent"
+                android:layout_height="wrap_content"
+                android:layout_marginTop="8dp"
+                android:visibility="gone" />
+
             <TextView
                 android:id="@+id/k2go_moddet_desc"
                 android:layout_width="match_parent"