From 9fd23c4c82a37fa1fdf90a7b60b844efa24e92b4 Mon Sep 17 00:00:00 2001
From: Joachim Schleicher <joachimsl@gmx.de>
Date: Thu, 28 Dec 2023 14:26:15 +0100
Subject: [PATCH] cleanup sharebox changes

* "data" moved to helptext
* drop css introduced in previous draft
* prettier.js
---
 umap/static/umap/base.css            |  4 +---
 umap/static/umap/js/umap.controls.js |  1 -
 umap/static/umap/js/umap.share.js    | 20 +++++++++++++++-----
 3 files changed, 16 insertions(+), 9 deletions(-)

diff --git a/umap/static/umap/base.css b/umap/static/umap/base.css
index b07dd8f19..13039fbdc 100644
--- a/umap/static/umap/base.css
+++ b/umap/static/umap/base.css
@@ -227,8 +227,7 @@ button.flat,
     font-size: 10px;
     border-radius: 0 2px;
 }
-.content .helptext,
-#umap-ui-container .help-text {
+.content .helptext {
     background-color: #eee;
     color: #000;
 }
@@ -410,7 +409,6 @@ input.switch:checked ~ label:after {
 .umap-multiplechoice.by5 {
     grid-template-columns: 1fr 1fr 1fr;
 }
-.button-bar.by4,
 .umap-multiplechoice.by4 {
     grid-template-columns: 1fr 1fr 1fr 1fr;
 }
diff --git a/umap/static/umap/js/umap.controls.js b/umap/static/umap/js/umap.controls.js
index 0af058ce9..019681078 100644
--- a/umap/static/umap/js/umap.controls.js
+++ b/umap/static/umap/js/umap.controls.js
@@ -1069,7 +1069,6 @@ L.U.TileLayerControl = L.Control.IconLayers.extend({
     L.Control.IconLayers.prototype.setLayers.call(this, layers.slice(0, maxShown))
     if (this.map.selected_tilelayer) this.setActiveLayer(this.map.selected_tilelayer)
   },
-
 })
 
 /* Used in edit mode to define the default tilelayer */
diff --git a/umap/static/umap/js/umap.share.js b/umap/static/umap/js/umap.share.js
index 7a397a499..75b1e5eed 100644
--- a/umap/static/umap/js/umap.share.js
+++ b/umap/static/umap/js/umap.share.js
@@ -55,7 +55,12 @@ L.U.Share = L.Class.extend({
       () => navigator.clipboard.writeText(this.mapUrl.value),
       this
     )
-    const mapUrlLabel = L.DomUtil.add('label', '', this.container, L._('Link to view the map'))
+    const mapUrlLabel = L.DomUtil.add(
+      'label',
+      '',
+      this.container,
+      L._('Link to view the map')
+    )
     this.mapUrl = L.DomUtil.create('input', 'umap-share-url', mapUrlLabel)
     this.mapUrl.type = 'text'
     this.mapUrl.readOnly = true
@@ -80,27 +85,32 @@ L.U.Share = L.Class.extend({
     L.DomUtil.create('hr', '', this.container)
 
     L.DomUtil.add('h4', '', this.container, L._('Download'))
-    L.DomUtil.add('small', 'label', this.container, L._('Only visible layers'))
+    L.DomUtil.add('small', 'label', this.container, L._("Only visible layers' data"))
     for (const key in this.EXPORT_TYPES) {
       if (this.EXPORT_TYPES.hasOwnProperty(key)) {
         L.DomUtil.createButton(
           'download-file',
           this.container,
-          this.EXPORT_TYPES[key].name || key + ' data',
+          this.EXPORT_TYPES[key].name || key,
           () => this.download(key),
           this
         )
       }
     }
     L.DomUtil.create('div', 'vspace', this.container)
-    L.DomUtil.add('small', 'label', this.container, L._('All data and settings of the map'))
+    L.DomUtil.add(
+      'small',
+      'label',
+      this.container,
+      L._('All data and settings of the map')
+    )
     const downloadUrl = L.Util.template(this.map.options.urls.map_download, {
       map_id: this.map.options.umap_id,
     })
     const link = L.DomUtil.createLink(
       'download-backup',
       this.container,
-      L._('Download full backup'),
+      L._('full backup'),
       downloadUrl
     )
     let name = this.map.options.name || 'data'