From 8f91278d1ada58b8c36294eb0a51122621d96566 Mon Sep 17 00:00:00 2001
From: Maxime NATUREL <maxime.naturel@niji.fr>
Date: Fri, 29 Jul 2022 15:58:18 +0200
Subject: [PATCH] Creating custom view for map loading error

---
 .../features/location/MapLoadingErrorView.kt  | 38 +++++++++++++++++
 .../res/layout/view_map_loading_error.xml     | 42 +++++++++++++++++++
 vector/src/main/res/values/strings.xml        |  4 +-
 3 files changed, 83 insertions(+), 1 deletion(-)
 create mode 100644 vector/src/main/java/im/vector/app/features/location/MapLoadingErrorView.kt
 create mode 100644 vector/src/main/res/layout/view_map_loading_error.xml

diff --git a/vector/src/main/java/im/vector/app/features/location/MapLoadingErrorView.kt b/vector/src/main/java/im/vector/app/features/location/MapLoadingErrorView.kt
new file mode 100644
index 0000000000..45b7dfd6d6
--- /dev/null
+++ b/vector/src/main/java/im/vector/app/features/location/MapLoadingErrorView.kt
@@ -0,0 +1,38 @@
+/*
+ * Copyright (c) 2022 New Vector Ltd
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package im.vector.app.features.location
+
+import android.content.Context
+import android.util.AttributeSet
+import android.view.LayoutInflater
+import androidx.constraintlayout.widget.ConstraintLayout
+import im.vector.app.databinding.ViewMapLoadingErrorBinding
+
+/**
+ * Custom view to display an error when map fails to load.
+ */
+class MapLoadingErrorView @JvmOverloads constructor(
+        context: Context, attrs: AttributeSet? = null, defStyleAttr: Int = 0
+) : ConstraintLayout(context, attrs, defStyleAttr) {
+
+    init {
+        ViewMapLoadingErrorBinding.inflate(
+                LayoutInflater.from(context),
+                this
+        )
+    }
+}
diff --git a/vector/src/main/res/layout/view_map_loading_error.xml b/vector/src/main/res/layout/view_map_loading_error.xml
new file mode 100644
index 0000000000..dca03bf49b
--- /dev/null
+++ b/vector/src/main/res/layout/view_map_loading_error.xml
@@ -0,0 +1,42 @@
+<?xml version="1.0" encoding="utf-8"?>
+<merge xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:tools="http://schemas.android.com/tools"
+    android:layout_width="match_parent"
+    android:layout_height="match_parent"
+    tools:parentTag="androidx.constraintlayout.widget.ConstraintLayout">
+
+    <androidx.constraintlayout.helper.widget.Flow
+        android:id="@+id/mapLoadingErrorContainer"
+        android:layout_width="0dp"
+        android:layout_height="0dp"
+        android:background="?vctr_system"
+        android:orientation="vertical"
+        app:constraint_referenced_ids="mapLoadingErrorIcon,mapLoadingErrorDescription"
+        app:flow_verticalGap="12dp"
+        app:flow_verticalStyle="packed"
+        app:layout_constraintBottom_toBottomOf="parent"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintTop_toTopOf="parent" />
+
+    <ImageView
+        android:id="@+id/mapLoadingErrorIcon"
+        android:layout_width="28dp"
+        android:layout_height="28dp"
+        android:contentDescription="@string/a11y_location_share_option_pinned_icon"
+        tools:src="@drawable/ic_warning_badge" />
+
+    <TextView
+        android:id="@+id/mapLoadingErrorDescription"
+        style="@style/TextAppearance.Vector.Body"
+        android:layout_width="0dp"
+        android:layout_height="wrap_content"
+        android:gravity="center"
+        android:text="@string/location_share_loading_map_error"
+        android:textColor="?vctr_content_primary"
+        app:layout_constraintEnd_toEndOf="parent"
+        app:layout_constraintStart_toStartOf="parent"
+        app:layout_constraintWidth_percent="0.8" />
+
+</merge>
diff --git a/vector/src/main/res/values/strings.xml b/vector/src/main/res/values/strings.xml
index 6b4eec1c9c..142cc3a8d9 100644
--- a/vector/src/main/res/values/strings.xml
+++ b/vector/src/main/res/values/strings.xml
@@ -3113,7 +3113,9 @@
     <string name="location_not_available_dialog_title">${app_name} could not access your location</string>
     <string name="location_not_available_dialog_content">${app_name} could not access your location. Please try again later.</string>
     <string name="location_share_external">Open with</string>
-    <string name="location_timeline_failed_to_load_map">Failed to load map</string>
+    <!-- TODO remove key -->
+    <string name="location_timeline_failed_to_load_map" tools:ignore="UnusedResources">Failed to load map</string>
+    <string name="location_share_loading_map_error">Unable to load map\nThis home server may not be configured to display maps.</string>
     <string name="location_share_live_enabled">Live location enabled</string>
     <string name="location_share_live_started">Loading live location…</string>
     <string name="location_share_live_ended">Live location ended</string>