From b249fdbab6a22401b1b00996f8eddf296fb7d07e Mon Sep 17 00:00:00 2001
From: cepprice <super.cepprice@gmail.com>
Date: Tue, 26 Jan 2021 20:09:25 +0500
Subject: [PATCH 1/2] Update UI of Description in Overview Tab in GPX context
 menu

---
 .../layout/dialog_edit_gpx_description.xml    |  63 +++++---
 .../layout/dialog_read_gpx_description.xml    |  42 +++--
 .../layout/gpx_description_preview_card.xml   | 147 ++++++++++++------
 .../osmand/plus/track/DescriptionCard.java    |  42 ++++-
 .../GpxEditDescriptionDialogFragment.java     |  61 ++++++--
 .../GpxReadDescriptionDialogFragment.java     |  25 +--
 6 files changed, 262 insertions(+), 118 deletions(-)

diff --git a/OsmAnd/res/layout/dialog_edit_gpx_description.xml b/OsmAnd/res/layout/dialog_edit_gpx_description.xml
index 52b5f97bdc..1d2428f82b 100644
--- a/OsmAnd/res/layout/dialog_edit_gpx_description.xml
+++ b/OsmAnd/res/layout/dialog_edit_gpx_description.xml
@@ -42,25 +42,31 @@
 			android:textSize="@dimen/dialog_header_text_size"
 			osmand:typeface="@string/font_roboto_medium" />
 
-		<net.osmand.plus.widgets.TextViewEx
-			android:id="@+id/btn_save"
+		<FrameLayout
 			android:layout_width="wrap_content"
 			android:layout_height="wrap_content"
 			android:layout_marginRight="@dimen/content_padding_half"
 			android:layout_marginEnd="@dimen/content_padding_half"
 			android:layout_gravity="center_vertical"
-			android:background="@drawable/btn_border_active"
-			android:paddingStart="@dimen/content_padding"
-			android:paddingLeft="@dimen/content_padding"
-			android:paddingTop="@dimen/content_padding_half"
-			android:paddingRight="@dimen/content_padding"
-			android:paddingEnd="@dimen/content_padding"
-			android:paddingBottom="@dimen/content_padding_half"
-			android:text="@string/shared_string_save"
-			android:textColor="?attr/active_color_basic"
-			android:textSize="@dimen/default_desc_text_size"
-			osmand:typeface="@string/font_roboto_medium" />
+			android:background="@drawable/btn_border_active">
 
+			<net.osmand.plus.widgets.TextViewEx
+				android:id="@+id/btn_save"
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:background="?attr/selectableItemBackground"
+				android:paddingStart="@dimen/content_padding"
+				android:paddingLeft="@dimen/content_padding"
+				android:paddingTop="@dimen/content_padding_half"
+				android:paddingRight="@dimen/content_padding"
+				android:paddingEnd="@dimen/content_padding"
+				android:paddingBottom="@dimen/content_padding_half"
+				android:text="@string/shared_string_save"
+				android:textColor="?attr/active_color_basic"
+				android:textSize="@dimen/default_desc_text_size"
+				osmand:typeface="@string/font_roboto_medium" />
+
+		</FrameLayout>
 
 	</LinearLayout>
 
@@ -69,17 +75,26 @@
 		android:layout_height="1dp"
 		android:background="?attr/divider_color" />
 
-	<net.osmand.plus.widgets.EditTextEx
-		android:id="@+id/description"
+	<ScrollView
 		android:layout_width="match_parent"
-		android:layout_height="wrap_content"
-		android:layout_margin="@dimen/content_padding"
-		android:layout_marginLeft="@dimen/content_padding"
-		android:layout_marginRight="@dimen/content_padding"
-		android:background="?attr/card_and_list_background_basic"
-		android:textColor="?android:attr/textColorPrimary"
-		android:textSize="@dimen/default_list_text_size"
-		osmand:typeface="@string/font_roboto_regular"
-		tools:text="Amsterdam is the Netherlands' capital and financial, cultural and creative centre with more than 850,000 inhabitants. Amsterdam is known for the canals that criss-cross the city, its impressive architecture and more than 1,500 bridges. The city has a heritage dating back to the Dutch Golden Age in the 17th century as well as a diverse art scene and a bustling nightlife." />
+		android:layout_height="match_parent">
+
+		<net.osmand.plus.widgets.EditTextEx
+			android:id="@+id/description"
+			android:layout_width="match_parent"
+			android:layout_height="wrap_content"
+			android:layout_marginStart="@dimen/content_padding"
+			android:layout_marginLeft="@dimen/content_padding"
+			android:layout_marginTop="@dimen/content_padding_half"
+			android:layout_marginRight="@dimen/content_padding"
+			android:layout_marginEnd="@dimen/content_padding"
+			android:layout_marginBottom="@dimen/content_padding_half"
+			android:background="?attr/card_and_list_background_basic"
+			android:textColor="?android:attr/textColorPrimary"
+			android:textSize="@dimen/default_list_text_size"
+			osmand:typeface="@string/font_roboto_regular"
+			tools:text="Amsterdam is the Netherlands' capital and financial, cultural and creative centre with more than 850,000 inhabitants. Amsterdam is known for the canals that criss-cross the city, its impressive architecture and more than 1,500 bridges. The city has a heritage dating back to the Dutch Golden Age in the 17th century as well as a diverse art scene and a bustling nightlife." />
+
+	</ScrollView>
 
 </LinearLayout>
\ No newline at end of file
diff --git a/OsmAnd/res/layout/dialog_read_gpx_description.xml b/OsmAnd/res/layout/dialog_read_gpx_description.xml
index f4cf390af5..d065868ab1 100644
--- a/OsmAnd/res/layout/dialog_read_gpx_description.xml
+++ b/OsmAnd/res/layout/dialog_read_gpx_description.xml
@@ -56,32 +56,40 @@
 				android:id="@+id/divider"
 				android:layout_width="match_parent"
 				android:layout_height="1dp"
-				android:background="?attr/divider_color"
+				android:background="?attr/ctx_menu_info_divider"
 				android:visibility="gone"
 				tools:visibility="visible"/>
 
-			<net.osmand.plus.widgets.TextViewEx
-				android:id="@+id/btn_edit"
+			<FrameLayout
 				android:layout_width="wrap_content"
 				android:layout_height="wrap_content"
 				android:layout_marginLeft="@dimen/content_padding"
 				android:layout_marginStart="@dimen/content_padding"
 				android:layout_marginTop="@dimen/context_menu_padding_margin_small"
 				android:layout_marginBottom="@dimen/context_menu_padding_margin_small"
-				android:gravity="center_vertical"
-				android:padding="@dimen/context_menu_padding_margin_small"
-				android:paddingStart="@dimen/context_menu_padding_margin_small"
-				android:paddingEnd="@dimen/context_menu_padding_margin_small"
-				android:drawablePadding="@dimen/list_content_padding_large"
-				osmand:drawableLeftCompat="@drawable/ic_action_edit_dark"
-				osmand:drawableStartCompat="@drawable/ic_action_edit_dark"
-				osmand:drawableTint="?attr/active_color_basic"
-				android:visibility="gone"
-				android:text="@string/shared_string_edit"
-				android:textColor="?attr/active_color_basic"
-				android:textSize="@dimen/default_list_text_size"
-				osmand:typeface="@string/font_roboto_medium"
-				tools:visibility="visible" />
+				android:background="@drawable/rounded_background_3dp">
+
+				<net.osmand.plus.widgets.TextViewEx
+					android:id="@+id/btn_edit"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:gravity="center_vertical"
+					android:background="?attr/selectableItemBackgroundBorderless"
+					android:padding="@dimen/context_menu_padding_margin_small"
+					android:paddingStart="@dimen/context_menu_padding_margin_small"
+					android:paddingEnd="@dimen/context_menu_padding_margin_small"
+					android:drawablePadding="@dimen/list_content_padding_large"
+					osmand:drawableLeftCompat="@drawable/ic_action_edit_dark"
+					osmand:drawableStartCompat="@drawable/ic_action_edit_dark"
+					osmand:drawableTint="?attr/active_color_basic"
+					android:visibility="gone"
+					android:text="@string/shared_string_edit"
+					android:textColor="?attr/active_color_basic"
+					android:textSize="@dimen/default_list_text_size"
+					osmand:typeface="@string/font_roboto_medium"
+					tools:visibility="visible" />
+
+			</FrameLayout>
 
 			<FrameLayout
 				android:id="@+id/bottom_empty_space"
diff --git a/OsmAnd/res/layout/gpx_description_preview_card.xml b/OsmAnd/res/layout/gpx_description_preview_card.xml
index d51c5a846e..a3b0ab8b25 100644
--- a/OsmAnd/res/layout/gpx_description_preview_card.xml
+++ b/OsmAnd/res/layout/gpx_description_preview_card.xml
@@ -40,69 +40,119 @@
 		tools:visibility="visible"
 		tools:src="@drawable/img_help_announcement_time_day"/>
 
-	<net.osmand.plus.widgets.TextViewEx
-		android:id="@+id/description"
+	<LinearLayout
+		android:id="@id/description_container"
 		android:layout_width="match_parent"
 		android:layout_height="wrap_content"
-		android:layout_marginStart="@dimen/content_padding"
-		android:layout_marginLeft="@dimen/content_padding"
-		android:layout_marginTop="@dimen/context_menu_padding_margin_small"
-		android:layout_marginRight="@dimen/content_padding"
-		android:layout_marginEnd="@dimen/content_padding"
-		android:textColor="?android:textColorPrimary"
-		android:textSize="@dimen/default_list_text_size"
-		android:lineSpacingMultiplier="1.1"
-		osmand:typeface="@string/font_roboto_regular"
-		tools:text="Amsterdam is the Netherlands' capital and financial, cultural and creative centre with more than 850,000 inhabitants. Amsterdam is known for the canals that criss-cross the city, its impressive architecture and more than 1,500 bridges. The city has a heritage dating back to the Dutch Golden Age in the 17th century as well as a diverse art scene and a bustling nightlife."/>
+		android:orientation="vertical">
+
+		<net.osmand.plus.widgets.TextViewEx
+			android:id="@+id/description"
+			android:layout_width="match_parent"
+			android:layout_height="wrap_content"
+			android:layout_marginStart="@dimen/content_padding"
+			android:layout_marginLeft="@dimen/content_padding"
+			android:layout_marginTop="@dimen/context_menu_padding_margin_small"
+			android:layout_marginRight="@dimen/content_padding"
+			android:layout_marginEnd="@dimen/content_padding"
+			android:textColor="?android:textColorPrimary"
+			android:textSize="@dimen/default_list_text_size"
+			android:lineSpacingMultiplier="1.1"
+			osmand:typeface="@string/font_roboto_regular"
+			tools:text="Amsterdam is the Netherlands' capital and financial, cultural and creative centre with more than 850,000 inhabitants. Amsterdam is known for the canals that criss-cross the city, its impressive architecture and more than 1,500 bridges. The city has a heritage dating back to the Dutch Golden Age in the 17th century as well as a diverse art scene and a bustling nightlife."/>
+
+		<FrameLayout
+			android:layout_width="match_parent"
+			android:layout_height="wrap_content"
+			android:layout_marginTop="@dimen/context_menu_padding_margin_small"
+			android:orientation="horizontal">
+
+			<FrameLayout
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:layout_marginStart="@dimen/context_menu_padding_margin_small"
+				android:layout_marginLeft="@dimen/context_menu_padding_margin_small"
+				android:background="@drawable/rounded_background_3dp">
+
+				<net.osmand.plus.widgets.TextViewEx
+					android:id="@+id/btn_read_full"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:layout_gravity="start"
+					android:gravity="center_vertical"
+					android:background="?attr/selectableItemBackgroundBorderless"
+					android:padding="@dimen/bottom_sheet_content_padding_small"
+					android:paddingStart="@dimen/bottom_sheet_content_padding_small"
+					android:paddingEnd="@dimen/bottom_sheet_content_padding_small"
+					android:drawablePadding="@dimen/bottom_sheet_content_padding_small"
+					osmand:drawableStartCompat="@drawable/ic_action_read_article"
+					osmand:drawableLeftCompat="@drawable/ic_action_read_article"
+					osmand:drawableTint="?attr/wikivoyage_active_color"
+					android:text="@string/context_menu_read_full"
+					android:textColor="?attr/wikivoyage_active_color"
+					android:textSize="@dimen/default_desc_text_size"
+					osmand:typeface="@string/font_roboto_medium" />
+
+			</FrameLayout>
+
+			<FrameLayout
+				android:layout_width="wrap_content"
+				android:layout_height="wrap_content"
+				android:layout_marginRight="@dimen/context_menu_padding_margin_small"
+				android:layout_marginEnd="@dimen/context_menu_padding_margin_small"
+				android:layout_gravity="end"
+				android:background="@drawable/rounded_background_3dp">
+
+				<net.osmand.plus.widgets.TextViewEx
+					android:id="@+id/btn_edit"
+					android:layout_width="wrap_content"
+					android:layout_height="wrap_content"
+					android:gravity="center_vertical"
+					android:background="?attr/selectableItemBackgroundBorderless"
+					android:padding="@dimen/bottom_sheet_content_padding_small"
+					android:paddingStart="@dimen/bottom_sheet_content_padding_small"
+					android:paddingEnd="@dimen/bottom_sheet_content_padding_small"
+					android:drawablePadding="@dimen/bottom_sheet_content_padding_small"
+					osmand:drawableRightCompat="@drawable/ic_action_edit_dark"
+					osmand:drawableEndCompat="@drawable/ic_action_edit_dark"
+					osmand:drawableTint="?attr/wikivoyage_active_color"
+					android:text="@string/shared_string_edit"
+					android:textColor="?attr/wikivoyage_active_color"
+					android:textSize="@dimen/default_desc_text_size"
+					osmand:typeface="@string/font_roboto_medium" />
+
+			</FrameLayout>
+
+		</FrameLayout>
+
+	</LinearLayout>
 
 	<FrameLayout
-		android:layout_width="match_parent"
+		android:id="@+id/btn_add"
+		android:layout_width="wrap_content"
 		android:layout_height="wrap_content"
-		android:layout_marginTop="@dimen/context_menu_padding_margin_small"
-		android:layout_marginBottom="@dimen/context_menu_padding_margin_small"
-		android:orientation="horizontal">
+		android:layout_marginStart="@dimen/card_padding"
+		android:layout_marginLeft="@dimen/card_padding"
+		android:layout_marginTop="@dimen/content_padding"
+		android:background="@drawable/rounded_background_3dp"
+		android:visibility="gone"
+		tools:visibility="visible">
 
 		<net.osmand.plus.widgets.TextViewEx
-			android:id="@+id/btn_read_full"
 			android:layout_width="wrap_content"
 			android:layout_height="wrap_content"
-			android:layout_marginStart="@dimen/context_menu_padding_margin_small"
-			android:layout_marginLeft="@dimen/context_menu_padding_margin_small"
-			android:layout_gravity="start"
-			android:gravity="center_vertical"
-			android:paddingStart="@dimen/bottom_sheet_content_padding_small"
-			android:paddingLeft="@dimen/bottom_sheet_content_padding_small"
-			android:paddingTop="@dimen/bottom_sheet_content_padding_small"
-			android:paddingRight="@dimen/bottom_sheet_content_padding_small"
-			android:paddingEnd="@dimen/bottom_sheet_content_padding_small"
-			android:paddingBottom="@dimen/bottom_sheet_content_padding_small"
-			android:drawablePadding="@dimen/bottom_sheet_content_padding_small"
-			osmand:drawableStartCompat="@drawable/ic_action_read_article"
-			osmand:drawableLeftCompat="@drawable/ic_action_read_article"
-			osmand:drawableTint="?attr/wikivoyage_active_color"
-			android:text="@string/context_menu_read_full"
-			android:textColor="?attr/wikivoyage_active_color"
-			android:textSize="@dimen/default_desc_text_size"
-			osmand:typeface="@string/font_roboto_medium" />
-
-		<net.osmand.plus.widgets.TextViewEx
-			android:id="@+id/btn_edit"
-			android:layout_width="wrap_content"
-			android:layout_height="wrap_content"
-			android:layout_marginRight="@dimen/context_menu_padding_margin_small"
-			android:layout_marginEnd="@dimen/context_menu_padding_margin_small"
-			android:layout_gravity="end"
 			android:gravity="center_vertical"
+			android:background="?attr/selectableItemBackgroundBorderless"
 			android:padding="@dimen/bottom_sheet_content_padding_small"
 			android:paddingStart="@dimen/bottom_sheet_content_padding_small"
 			android:paddingEnd="@dimen/bottom_sheet_content_padding_small"
-			android:drawablePadding="@dimen/bottom_sheet_content_padding_small"
-			osmand:drawableRightCompat="@drawable/ic_action_edit_dark"
-			osmand:drawableEndCompat="@drawable/ic_action_edit_dark"
+			android:drawablePadding="@dimen/dialog_button_height"
+			osmand:drawableStartCompat="@drawable/ic_action_add"
+			osmand:drawableLeftCompat="@drawable/ic_action_add"
 			osmand:drawableTint="?attr/wikivoyage_active_color"
-			android:text="@string/shared_string_edit"
+			android:text="@string/add_description"
 			android:textColor="?attr/wikivoyage_active_color"
-			android:textSize="@dimen/default_desc_text_size"
+			android:textSize="@dimen/default_list_text_size"
 			osmand:typeface="@string/font_roboto_medium" />
 
 	</FrameLayout>
@@ -110,6 +160,7 @@
 	<FrameLayout
 		android:layout_width="match_parent"
 		android:layout_height="wrap_content"
+		android:layout_marginTop="@dimen/context_menu_padding_margin_small"
 		android:background="?attr/activity_background_basic">
 
 		<include
diff --git a/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java b/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java
index 0eea4abd1c..1280a183ef 100644
--- a/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java
+++ b/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java
@@ -1,14 +1,14 @@
 package net.osmand.plus.track;
 
 import android.view.View;
-
-import androidx.annotation.NonNull;
-import androidx.appcompat.widget.AppCompatImageView;
+import android.widget.FrameLayout;
+import android.widget.LinearLayout;
 
 import com.squareup.picasso.Callback;
 import com.squareup.picasso.Picasso;
 import com.squareup.picasso.RequestCreator;
 
+import net.osmand.GPXUtilities;
 import net.osmand.GPXUtilities.GPXFile;
 import net.osmand.PicassoUtils;
 import net.osmand.plus.R;
@@ -19,6 +19,9 @@ import net.osmand.plus.widgets.TextViewEx;
 import net.osmand.plus.wikipedia.WikiArticleHelper;
 import net.osmand.util.Algorithms;
 
+import androidx.annotation.NonNull;
+import androidx.appcompat.widget.AppCompatImageView;
+
 import static net.osmand.plus.myplaces.TrackActivityFragmentAdapter.getMetadataImageLink;
 
 public class DescriptionCard extends BaseCard {
@@ -37,11 +40,8 @@ public class DescriptionCard extends BaseCard {
 
 	@Override
 	protected void updateContent() {
-		if (gpxFile.metadata == null || gpxFile.metadata.getDescription() == null) {
-			AndroidUiHelper.updateVisibility(view, false);
-			return;
-		} else {
-			AndroidUiHelper.updateVisibility(view, true);
+		if (gpxFile.metadata == null) {
+			gpxFile.metadata = new GPXUtilities.Metadata();
 		}
 
 		final String title = gpxFile.metadata.getArticleTitle();
@@ -50,6 +50,31 @@ public class DescriptionCard extends BaseCard {
 
 		setupImage(imageUrl);
 
+		if (descriptionHtml == null || Algorithms.isEmpty(descriptionHtml.trim())) {
+			showAddBtn();
+		} else {
+			showDescription(title, imageUrl, descriptionHtml);
+		}
+	}
+
+	private void showAddBtn() {
+		LinearLayout descriptionContainer = view.findViewById(R.id.description_container);
+		FrameLayout addBtn = view.findViewById(R.id.btn_add);
+
+		addBtn.setOnClickListener(new View.OnClickListener() {
+			@Override
+			public void onClick(View v) {
+				GpxEditDescriptionDialogFragment.showInstance(getMapActivity(), "", null);
+			}
+		});
+		AndroidUiHelper.updateVisibility(descriptionContainer, false);
+		AndroidUiHelper.updateVisibility(addBtn, true);
+	}
+
+	private void showDescription(final String title, final String imageUrl, final String descriptionHtml) {
+		LinearLayout descriptionContainer = view.findViewById(R.id.description_container);
+		AndroidUiHelper.updateVisibility(descriptionContainer, true);
+
 		TextViewEx tvDescription = view.findViewById(R.id.description);
 		tvDescription.setText(getFirstParagraph(descriptionHtml));
 
@@ -60,6 +85,7 @@ public class DescriptionCard extends BaseCard {
 				GpxReadDescriptionDialogFragment.showInstance(mapActivity, title, imageUrl, descriptionHtml);
 			}
 		});
+
 		TextViewEx editBtn = view.findViewById(R.id.btn_edit);
 		editBtn.setOnClickListener(new View.OnClickListener() {
 			@Override
diff --git a/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java b/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java
index ca868bc3bc..d17cd91667 100644
--- a/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java
+++ b/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java
@@ -1,5 +1,7 @@
 package net.osmand.plus.track;
 
+import android.content.Context;
+import android.content.DialogInterface;
 import android.os.AsyncTask;
 import android.os.Bundle;
 import android.text.Editable;
@@ -7,12 +9,6 @@ import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
 
-import androidx.annotation.NonNull;
-import androidx.annotation.Nullable;
-import androidx.fragment.app.Fragment;
-import androidx.fragment.app.FragmentActivity;
-import androidx.fragment.app.FragmentManager;
-
 import net.osmand.GPXUtilities.GPXFile;
 import net.osmand.PlatformUtil;
 import net.osmand.plus.OsmandApplication;
@@ -22,12 +18,18 @@ import net.osmand.plus.activities.MapActivity;
 import net.osmand.plus.base.BaseOsmAndDialogFragment;
 import net.osmand.plus.track.SaveGpxAsyncTask.SaveGpxListener;
 import net.osmand.plus.widgets.EditTextEx;
-import net.osmand.util.Algorithms;
 
 import org.apache.commons.logging.Log;
 
 import java.io.File;
 
+import androidx.annotation.NonNull;
+import androidx.annotation.Nullable;
+import androidx.appcompat.app.AlertDialog;
+import androidx.fragment.app.Fragment;
+import androidx.fragment.app.FragmentActivity;
+import androidx.fragment.app.FragmentManager;
+
 public class GpxEditDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 
 	public static final String TAG = GpxEditDescriptionDialogFragment.class.getSimpleName();
@@ -37,6 +39,8 @@ public class GpxEditDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 
 	private EditTextEx editableHtml;
 
+	private String htmlCode;
+
 	@Nullable
 	@Override
 	public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {
@@ -46,11 +50,16 @@ public class GpxEditDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 		View view = themedInflater.inflate(R.layout.dialog_edit_gpx_description, container, false);
 
 		editableHtml = view.findViewById(R.id.description);
+		editableHtml.requestFocus();
 
 		view.findViewById(R.id.btn_close).setOnClickListener(new View.OnClickListener() {
 			@Override
 			public void onClick(View v) {
-				dismiss();
+				if (shouldClose()) {
+					dismiss();
+				} else {
+					showDismissDialog();
+				}
 			}
 		});
 
@@ -58,7 +67,7 @@ public class GpxEditDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 			@Override
 			public void onClick(View v) {
 				Editable editable = editableHtml.getText();
-				if (!Algorithms.isEmpty(editable) && !saveGpx(editable.toString())) {
+				if (editable != null && !saveGpx(editable.toString())) {
 					dismiss();
 				}
 			}
@@ -66,15 +75,43 @@ public class GpxEditDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 
 		Bundle args = getArguments();
 		if (args != null) {
-			String html = args.getString(CONTENT_KEY);
-			if (html != null) {
-				editableHtml.setText(html);
+			htmlCode = args.getString(CONTENT_KEY);
+			if (htmlCode != null) {
+				editableHtml.setText(htmlCode);
 			}
 		}
 
 		return view;
 	}
 
+
+
+	private boolean shouldClose() {
+		Editable editable = editableHtml.getText();
+		if (htmlCode == null || editable == null || editable.toString() == null) {
+			return true;
+		}
+		if (htmlCode.equals(editable.toString())) {
+			return true;
+		}
+		return false;
+	}
+
+	private void showDismissDialog() {
+		Context themedContext = UiUtilities.getThemedContext(getMapActivity(), isNightMode(false));
+		AlertDialog.Builder dismissDialog = new AlertDialog.Builder(themedContext);
+		dismissDialog.setTitle(getString(R.string.shared_string_dismiss));
+		dismissDialog.setMessage(getString(R.string.exit_without_saving));
+		dismissDialog.setNegativeButton(R.string.shared_string_cancel, null);
+		dismissDialog.setPositiveButton(R.string.shared_string_exit, new DialogInterface.OnClickListener() {
+			@Override
+			public void onClick(DialogInterface dialog, int which) {
+				dismiss();
+			}
+		});
+		dismissDialog.show();
+	}
+
 	private boolean saveGpx(final String html) {
 		MapActivity mapActivity = getMapActivity();
 		if (mapActivity == null || mapActivity.getTrackMenuFragment() == null) {
diff --git a/OsmAnd/src/net/osmand/plus/track/GpxReadDescriptionDialogFragment.java b/OsmAnd/src/net/osmand/plus/track/GpxReadDescriptionDialogFragment.java
index 42a4441cdc..b652c7a3bd 100644
--- a/OsmAnd/src/net/osmand/plus/track/GpxReadDescriptionDialogFragment.java
+++ b/OsmAnd/src/net/osmand/plus/track/GpxReadDescriptionDialogFragment.java
@@ -3,6 +3,7 @@ package net.osmand.plus.track;
 import android.content.Context;
 import android.graphics.Color;
 import android.graphics.drawable.Drawable;
+import android.os.Build;
 import android.os.Bundle;
 import android.util.Base64;
 import android.view.LayoutInflater;
@@ -15,14 +16,6 @@ import android.webkit.WebSettings;
 import android.webkit.WebView;
 import android.webkit.WebViewClient;
 
-import androidx.annotation.NonNull;
-import androidx.annotation.Nullable;
-import androidx.appcompat.widget.AppCompatImageView;
-import androidx.appcompat.widget.Toolbar;
-import androidx.core.content.ContextCompat;
-import androidx.fragment.app.FragmentActivity;
-import androidx.fragment.app.FragmentManager;
-
 import com.squareup.picasso.Callback;
 import com.squareup.picasso.Picasso;
 import com.squareup.picasso.RequestCreator;
@@ -39,6 +32,14 @@ import net.osmand.plus.widgets.WebViewEx;
 import net.osmand.plus.wikivoyage.WikivoyageUtils;
 import net.osmand.util.Algorithms;
 
+import androidx.annotation.NonNull;
+import androidx.annotation.Nullable;
+import androidx.appcompat.widget.AppCompatImageView;
+import androidx.appcompat.widget.Toolbar;
+import androidx.core.content.ContextCompat;
+import androidx.fragment.app.FragmentActivity;
+import androidx.fragment.app.FragmentManager;
+
 public class GpxReadDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 
 	public static final String TAG = GpxReadDescriptionDialogFragment.class.getSimpleName();
@@ -181,10 +182,16 @@ public class GpxReadDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 
 	private void setupWebView(final View view) {
 		webView = view.findViewById(R.id.content);
+		webView.getSettings().setCacheMode(WebSettings.LOAD_NO_CACHE);
+		if (Build.VERSION.SDK_INT >= 19) {
+			webView.setLayerType(View.LAYER_TYPE_HARDWARE, null);
+		}
+		else {
+			webView.setLayerType(View.LAYER_TYPE_SOFTWARE, null);
+		}
 		webView.setScrollbarFadingEnabled(true);
 		webView.setVerticalScrollBarEnabled(false);
 		webView.setBackgroundColor(Color.TRANSPARENT);
-		webView.setLayerType(WebView.LAYER_TYPE_HARDWARE, null);
 		webView.getSettings().setTextZoom((int) (getResources().getConfiguration().fontScale * 100f));
 		webView.getSettings().setDomStorageEnabled(true);
 		webView.getSettings().setLoadWithOverviewMode(true);

From 27dd3423754914ca884140b41fbd4cd3e9960645 Mon Sep 17 00:00:00 2001
From: cepprice <super.cepprice@gmail.com>
Date: Wed, 27 Jan 2021 15:48:10 +0500
Subject: [PATCH 2/2] Small edit and fix

---
 OsmAnd/src/net/osmand/plus/track/DescriptionCard.java      | 5 ++++-
 .../plus/track/GpxEditDescriptionDialogFragment.java       | 7 +------
 2 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java b/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java
index 1280a183ef..8195c8d417 100644
--- a/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java
+++ b/OsmAnd/src/net/osmand/plus/track/DescriptionCard.java
@@ -50,7 +50,7 @@ public class DescriptionCard extends BaseCard {
 
 		setupImage(imageUrl);
 
-		if (descriptionHtml == null || Algorithms.isEmpty(descriptionHtml.trim())) {
+		if (Algorithms.isBlank(descriptionHtml)) {
 			showAddBtn();
 		} else {
 			showDescription(title, imageUrl, descriptionHtml);
@@ -73,7 +73,10 @@ public class DescriptionCard extends BaseCard {
 
 	private void showDescription(final String title, final String imageUrl, final String descriptionHtml) {
 		LinearLayout descriptionContainer = view.findViewById(R.id.description_container);
+		FrameLayout addBtn = view.findViewById(R.id.btn_add);
+
 		AndroidUiHelper.updateVisibility(descriptionContainer, true);
+		AndroidUiHelper.updateVisibility(addBtn, false);
 
 		TextViewEx tvDescription = view.findViewById(R.id.description);
 		tvDescription.setText(getFirstParagraph(descriptionHtml));
diff --git a/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java b/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java
index d17cd91667..a79a392f8e 100644
--- a/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java
+++ b/OsmAnd/src/net/osmand/plus/track/GpxEditDescriptionDialogFragment.java
@@ -84,17 +84,12 @@ public class GpxEditDescriptionDialogFragment extends BaseOsmAndDialogFragment {
 		return view;
 	}
 
-
-
 	private boolean shouldClose() {
 		Editable editable = editableHtml.getText();
 		if (htmlCode == null || editable == null || editable.toString() == null) {
 			return true;
 		}
-		if (htmlCode.equals(editable.toString())) {
-			return true;
-		}
-		return false;
+		return htmlCode.equals(editable.toString());
 	}
 
 	private void showDismissDialog() {