From 0c920c38fcc6b53592eaa625833b8dd2283e927c Mon Sep 17 00:00:00 2001
From: Levin Li <lilinfeng303@outlook.com>
Date: Sun, 18 Dec 2022 14:49:24 +0800
Subject: [PATCH] Elevate titles

---
 .../favorite/DestinationDetailFragment.kt     |  8 +++++--
 .../mobilecelestia/info/InfoFragment.kt       | 16 ++++++++++++-
 .../layout/fragment_destination_detail.xml    | 23 ++++---------------
 3 files changed, 26 insertions(+), 21 deletions(-)

diff --git a/app/src/main/java/space/celestia/mobilecelestia/favorite/DestinationDetailFragment.kt b/app/src/main/java/space/celestia/mobilecelestia/favorite/DestinationDetailFragment.kt
index 921d4564..43f03189 100644
--- a/app/src/main/java/space/celestia/mobilecelestia/favorite/DestinationDetailFragment.kt
+++ b/app/src/main/java/space/celestia/mobilecelestia/favorite/DestinationDetailFragment.kt
@@ -44,11 +44,9 @@ class DestinationDetailFragment : NavigationFragment.SubFragment() {
         savedInstanceState: Bundle?
     ): View? {
         val view = inflater.inflate(R.layout.fragment_destination_detail, container, false)
-        val title = view.findViewById<TextView>(R.id.title)
         val content = view.findViewById<TextView>(R.id.description)
         val button = view.findViewById<Button>(R.id.action_button)
 
-        title.text = item?.name ?: ""
         content.text = item?.description ?: ""
 
         button.setOnClickListener { this.onButtonClick() }
@@ -64,6 +62,12 @@ class DestinationDetailFragment : NavigationFragment.SubFragment() {
         return view
     }
 
+    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
+        super.onViewCreated(view, savedInstanceState)
+
+        title = item?.name ?: ""
+    }
+
     private fun onButtonClick() {
         val destination = item ?: return
         listener?.onGoToDestination(destination)
diff --git a/app/src/main/java/space/celestia/mobilecelestia/info/InfoFragment.kt b/app/src/main/java/space/celestia/mobilecelestia/info/InfoFragment.kt
index 724c5664..aa5bf511 100644
--- a/app/src/main/java/space/celestia/mobilecelestia/info/InfoFragment.kt
+++ b/app/src/main/java/space/celestia/mobilecelestia/info/InfoFragment.kt
@@ -22,6 +22,7 @@ import android.view.ViewGroup
 import android.widget.TextView
 import androidx.core.view.ViewCompat
 import androidx.core.view.WindowInsetsCompat
+import androidx.core.view.isVisible
 import androidx.core.view.updatePadding
 import androidx.lifecycle.lifecycleScope
 import androidx.recyclerview.widget.GridLayoutManager
@@ -60,6 +61,8 @@ class InfoFragment : NavigationFragment.SubFragment() {
     @Inject
     lateinit var appCore: AppCore
 
+    private lateinit var objectName: String
+
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
 
@@ -79,13 +82,24 @@ class InfoFragment : NavigationFragment.SubFragment() {
         titleLabel = view.findViewById(R.id.title)
         contentLabel = view.findViewById(R.id.content)
         linkView = view.findViewById(R.id.link_preview)
-        titleLabel.text = appCore.simulation.universe.getNameForSelection(selection)
+        objectName = appCore.simulation.universe.getNameForSelection(selection)
+        if (embeddedInNavigation)
+            titleLabel.isVisible = false
+        else
+            titleLabel.text = objectName
         contentLabel.text = appCore.getOverviewForSelection(selection)
 
         reload(true)
         return view
     }
 
+    override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
+        super.onViewCreated(view, savedInstanceState)
+
+        if (embeddedInNavigation)
+            title = objectName
+    }
+
     override fun onAttach(context: Context) {
         super.onAttach(context)
         if (context is Listener) {
diff --git a/app/src/main/res/layout/fragment_destination_detail.xml b/app/src/main/res/layout/fragment_destination_detail.xml
index 0462da44..b6c365d6 100644
--- a/app/src/main/res/layout/fragment_destination_detail.xml
+++ b/app/src/main/res/layout/fragment_destination_detail.xml
@@ -25,24 +25,11 @@
             android:clipToPadding="false"
             android:paddingTop="@dimen/common_page_medium_margin_vertical"
             android:layout_height="0dp">
-            <LinearLayout
-                android:orientation="vertical"
-                android:layout_width="match_parent"
-                android:layout_height="wrap_content">
-
-                <TextView
-                    android:id="@+id/title"
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content"
-                    android:layout_marginBottom="@dimen/common_page_medium_gap_vertical"
-                    style="@style/HeadlineLabel"/>
-
-                <TextView
-                    android:id="@+id/description"
-                    style="@style/BodyLabel"
-                    android:layout_width="wrap_content"
-                    android:layout_height="wrap_content" />
-            </LinearLayout>
+            <TextView
+                android:id="@+id/description"
+                style="@style/BodyLabel"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content" />
         </androidx.core.widget.NestedScrollView>
 
         <Space