Merge f4ae51ae7e into 190cac3105

umap/static/umap/base.css

@@ -3,7 +3,9 @@
 */
 body, div, ul, ol, li, a, section, nav,
 h1, h2, h3, h4, h5, h6, label,
-hr, input, textarea, select {
+hr, input, textarea {
+    -moz-box-sizing:border-box;
+    -webkit-box-sizing:border-box;
     box-sizing: border-box;
     margin: 0;
     padding: 0;
@@ -17,11 +19,7 @@ h1, h2, h3, h4, h5, h6, label, hr {
 }
 a {
     text-decoration: none;
-    color: var(--link-color);
+    color: #1F5C39;
-}
-a.main {
-    font-weight: bold;
-    text-decoration: underline;
 }
 a[href^="http"]:not(nav.umap-nav a) {
     text-decoration: underline;
@@ -83,6 +81,9 @@ hgroup > * {
 hgroup {
     margin-bottom: var(--box-margin);
 }
+hgroup > :not(:first-child):last-child {
+    font-weight: normal;
+}
 hgroup p,
 hgroup button {
     margin: 0;
@@ -201,10 +202,6 @@ dt {
 /*   Various   */
 /* *********** */
 
-.nobr {
-    white-space: nowrap;
-}
-
 .text {
     word-break: break-word;
     white-space: pre-line;

umap/static/umap/content.css

@@ -19,6 +19,7 @@ input:-moz-placeholder, :-moz-placeholder {
 .search-form {
     display: flex;
     align-items: baseline;
+    gap: calc(var(--gutter) / 2);
     max-width: 800px;
     margin: 0 auto;
 }
@@ -185,8 +186,6 @@ h2.tabs a:hover {
     white-space: nowrap;
     text-overflow: ellipsis;
     vertical-align: middle;
-    font-size: small;
-    padding: 0 3px;
 }
 .card {
     border: 1px solid var(--color-lightGray);
@@ -214,6 +213,7 @@ h2.tabs a:hover {
 /*            colors            */
 /* **************************** */
 
+input[type="submit"],
 .button {
     background-color: var(--button-primary-background);
     color: var(--button-primary-color);
@@ -405,9 +405,22 @@ html[dir="rtl"] .content .icon-delete {
         align-items: center;
     }
 }
+.table-header form input {
+    border: 2px solid var(--color-darkBlue);
+    border-radius: 0;
+    padding: .5rem 1rem;
+    margin-bottom: 0;
+    line-height: inherit;
+    height: 40px;
+}
 .table-header form input[type="search"] {
     width: 30ch;
 }
+.table-header form input[type="submit"] {
+    background-color: var(--color-darkBlue);
+    color: white;
+    font-weight: bold;
+}
 
 .table-header .button-download {
     width: inherit;

umap/static/umap/css/bar.css

@@ -4,7 +4,6 @@
     background-color: var(--color-darkGray);
     width: auto;
     margin-bottom: 0;
-    min-height: initial;
 }
 .umap-main-edit-toolbox [type=button]:hover {
     text-decoration: underline;
@@ -14,14 +13,9 @@
     padding: 0 var(--text-margin);
     background-color: inherit;
 }
-.leaflet-container .edit-undo {
-    margin-left: var(--box-margin);
-}
-.leaflet-container .edit-undo[disabled],
-.leaflet-container .edit-redo[disabled] {
-    filter: invert(.5);
-}
 .leaflet-container .edit-save,
+.leaflet-container .edit-undo,
+.leaflet-container .edit-redo,
 .leaflet-container .edit-disable,
 .leaflet-container .connected-peers
 {
@@ -46,6 +40,8 @@
     color: var(--color-darkGray);
 }
 
+.leaflet-container .edit-undo:hover,
+.leaflet-container .edit-redo:hover,
 .leaflet-container .edit-disable:hover {
     border: 0.5px solid rgba(153, 153, 153, 0.80);
     text-decoration: none;

umap/static/umap/css/form.css

@@ -7,15 +7,16 @@ input[type="datetime-local"], input[type="email"], input[type="number"],
 input[type="search"], input[type="tel"], input[type="time"], input[type="file"],
 input[type="url"], textarea {
     background-color: white;
-    border: 2px solid var(--color-darkBlue);
+    border: 1px solid #CCCCCC;
-    color: var(--text-color);
+    border-radius: 2px 2px 2px 2px;
+    box-shadow: 0 1px 2px rgba(0, 0, 0, 0.1) inset;
+    color: rgba(0, 0, 0, 0.75);
     display: block;
     font-family: inherit;
     margin: 0;
-    padding: .5rem 1rem;
+    margin-bottom: var(--box-margin);
+    padding: 7px;
     width: 100%;
-    line-height: inherit;
-    min-height: 40px;
 }
 input[type="range"] {
     margin-top: 10px;
@@ -46,15 +47,11 @@ input[type=checkbox]:checked:after {
     content: '✓';
     color: var(--color-darkGray);
 }
+
 input[data-modified=true] {
     background-color: var(--color-lightCyan);
     border: 1px solid var(--color-darkGray);
 }
-input + select,
-select + input,
-input + input {
-    border-left: none;
-}
 textarea {
     height: inherit;
     padding: 7px;
@@ -62,18 +59,13 @@ textarea {
     min-height: 6rlh;
 }
 select {
-    border: 2px solid var(--color-darkBlue);
+    border: 1px solid #222;
     width: 100%;
     padding: var(--button-padding);
-    background-color: var(--background-color);
-    color: var(--text-color);
-    min-height: 40px;
 }
 .dark select {
-    color: var(--text-color);
+    color: #efefef;
-    background-color: var(--color-darkGray);
+    background-color: #393F3F;
-    border-color: var(--color-dark);
-    border-width: 1px;
 }
 select[multiple="multiple"] {
     height: auto;
@@ -83,16 +75,16 @@ select[multiple="multiple"] {
 input[type="submit"] {
     display: flex;
     align-items: center;
+    margin-bottom: 14px;
     text-align: center;
+    border-radius: 2px;
+    font-weight: normal;
     cursor: pointer;
     padding: var(--button-padding);
     border: none;
     text-decoration: none;
+    background-color: white;
     justify-content: center;
-    background-color: var(--color-darkBlue);
-    color: white;
-    font-weight: normal;
-    min-height: 40px;
 }
 .dark .button,
 .dark [type="button"] {
@@ -100,9 +92,6 @@ input[type="submit"] {
     color: var(--text-color);
     border: 1px solid #1b1f20;
 }
-.button.primary {
-    font-weight: bold;
-}
 .dark .button.primary:not([disabled]),
 .dark [type="button"].primary:not([disabled]) {
     background-color: var(--color-brightCyan);
@@ -171,14 +160,6 @@ input + .help-text {
 .formbox.with-switch {
     padding-top: 2px;
 }
-.with-switch {
-    overflow: hidden;
-}
-.formbox select,
-.formbox textarea,
-.formbox input {
-    margin-bottom: var(--text-margin);
-}
 fieldset.formbox {
     border: none;
     border-top: 1px solid var(--color-lightGray);
@@ -219,9 +200,8 @@ input[value]:invalid {
     background-color: darkred;
 }
 .dark input, .dark textarea {
-    background-color: var(--color-darkerGray);
+    background-color: #232729;
-    border-color: var(--color-dark);
+    border-color: #1b1f20;
-    border-width: 1px;
     color: #efefef;
 }
 details {
@@ -297,6 +277,9 @@ input.switch:empty ~ label {
     text-indent: 6em;
     margin: 0.2em 0;
     cursor: pointer;
+    -webkit-user-select: none;
+    -moz-user-select: none;
+    -ms-user-select: none;
     user-select: none;
     text-shadow: 0 1px rgba(0, 0, 0, 0.1);
     width: 80px;
@@ -584,7 +567,6 @@ input.blur {
     vertical-align: middle;
     border-start-end-radius: 0;
     border-end-end-radius: 0;
-    margin-bottom: 0;
 }
 .blur + .button,
 .blur + [type="button"] {

umap/static/umap/js/modules/browser.js

@@ -61,7 +61,7 @@ export default class Browser {
     DomEvent.on(zoom_to, 'click', viewFeature)
     DomEvent.on(title, 'click', viewFeature)
     DomEvent.on(edit, 'click', feature.edit, feature)
-    DomEvent.on(del, 'click', feature.del, feature)
+    DomEvent.on(del, 'click', feature.confirmDelete, feature)
     // HOTFIX. Remove when this is released:
     // https://github.com/Leaflet/Leaflet/pull/9052
     DomEvent.disableClickPropagation(row)

umap/static/umap/js/modules/data/features.js

@@ -278,8 +278,7 @@ class Feature {
         <i class="icon icon-24 icon-delete"></i>${translate('Delete')}
       </button>`)
     button.addEventListener('click', () => {
-      this.del()
+      this.confirmDelete().then(() => this._umap.editPanel.close())
-      this._umap.editPanel.close()
     })
     container.appendChild(button)
   }
@@ -364,6 +363,17 @@ class Feature {
     return popup.loadContent()
   }
 
+  async confirmDelete() {
+    const confirmed = await this._umap.dialog.confirm(
+      translate('Are you sure you want to delete the feature?')
+    )
+    if (confirmed) {
+      this.del()
+      return true
+    }
+    return false
+  }
+
   del(sync) {
     this.isDirty = true
     this._umap._leafletMap.closePopup()
@@ -508,7 +518,7 @@ class Feature {
         icon: 'icon-edit',
       },
       {
-        action: () => this.del(),
+        action: () => this.confirmDelete(),
         title: translate('Delete this feature'),
         icon: 'icon-delete',
       },
@@ -663,7 +673,7 @@ class Feature {
       },
       {
         label: translate('Delete this feature'),
-        action: () => this.del(),
+        action: () => this.confirmDelete(),
       },
       {
         label: translate('Clone this feature'),

umap/static/umap/js/modules/data/layer.js

@@ -1237,14 +1237,23 @@ export class DataLayer {
       translate('Delete layer')
     )
     if (this.isReadOnly()) {
-      container.classList.add('readonly')
+      DomUtil.addClass(container, 'readonly')
     } else {
-      edit.addEventListener('click', () => this.edit())
+      DomEvent.on(edit, 'click', this.edit, this)
-      table.addEventListener('click', () => this.tableEdit())
+      DomEvent.on(table, 'click', this.tableEdit, this)
-      remove.addEventListener('click', () => {
+      DomEvent.on(
+        remove,
+        'click',
+        function () {
           if (!this.isVisible()) return
+          this._umap.dialog
+            .confirm(translate('Are you sure you want to delete this layer?'))
+            .then(() => {
               this.del()
             })
+        },
+        this
+      )
     }
     DomEvent.on(toggle, 'click', () => this.toggle())
     DomEvent.on(zoomTo, 'click', this.zoomTo, this)

umap/static/umap/js/modules/form/fields.js

@@ -152,9 +152,7 @@ Fields.Textarea = class extends BaseElement {
       this.textarea.value = value
     }
   }
-  clear() {
+
-    this.textarea.value = ''
-  }
   value() {
     return this.textarea.value
   }

umap/static/umap/js/modules/help.js

@@ -196,7 +196,7 @@ export default class Help {
         .split('+')
         .map((el) => `<kbd>${el}</kbd>`)
         .join('+')
-      label += ` <span class="nobr">${shortcut}</span>`
+      label += ` ${shortcut}`
     } else {
       label += ` (${shortcut})`
     }

umap/static/umap/js/modules/ui/bar.js

@@ -11,11 +11,13 @@ const TOP_BAR_TEMPLATE = `
         <div class="logo"><a class="" href="/" title="${translate('Go to the homepage')}">uMap</a></div>
         <button class="map-name flat truncate" type="button" data-ref="name"></button>
         <button class="share-status flat truncate" type="button" data-ref="share"></button>
-        <button class="edit-undo round flat" type="button" data-ref="undo" disabled>
+        <button class="edit-undo round" type="button" data-ref="undo" disabled>
             <i class="icon icon-16 icon-undo"></i>
+            <span>${translate('Undo')}</span>
         </button>
-        <button class="edit-redo round flat" type="button" data-ref="redo" disabled>
+        <button class="edit-redo round" type="button" data-ref="redo" disabled>
             <i class="icon icon-16 icon-redo"></i>
+            <span>${translate('Redo')}</span>
         </button>
     </div>
     <div class="umap-right-edit-toolbox" data-ref="right">

umap/static/umap/js/modules/utils.js

@@ -126,7 +126,6 @@ export function escapeHTML(s) {
       'frameborder',
       'scrolling',
       'controls',
-      'class',
     ],
     ALLOWED_ATTR: ['href', 'src', 'width', 'height', 'style', 'dir', 'title', 'type'],
     // Added: `geo:` URL scheme as defined in RFC5870:

umap/static/umap/map.css

@@ -37,6 +37,7 @@ html[dir="rtl"] .leaflet-tooltip-pane > * {
     background-color: var(--color-lightGray);
 }
 
+
 /* *********** */
 /* Structure   */
 /* *********** */
@@ -93,8 +94,6 @@ html[dir="rtl"] .leaflet-tooltip-pane > * {
     background-image: url('./img/24.svg');
     text-indent: -9999px;
     margin-bottom: 0;
-    background-color: white;
-    min-height: initial;
 }
 .leaflet-control.display-on-more,
 .umap-control-less {
@@ -450,7 +449,7 @@ ul.photon-autocomplete {
     display: inline-block;
     width: 16px;
     height: 16px;
-    margin-inline-start: 2px;
+    margin-inline-start: 5px;
     background-position: 2px -4px;
     background-repeat: no-repeat;
     background-image: url('./img/16.svg');
@@ -471,7 +470,6 @@ ul.photon-autocomplete {
 }
 .umap-getstarted button {
     width: 100%;
-    margin-bottom: var(--text-margin);
 }
 .umap-help {
     font-style: italic;
@@ -662,10 +660,6 @@ a.umap-control-caption,
 .umap-caption .header i.icon {
     flex-shrink: 0;
 }
-.umap-caption button {
-    background-color: var(--background-color);
-    color: var(--text-color);
-}
 .umap-browser .main-toolbox {
     padding-left: 4px; /* Align with toolbox below  */
     border-top: 1px solid var(--color-mediumGray);

umap/static/umap/vars.css

@@ -1,5 +1,6 @@
 :root {
     /* Colors. */
+    --color-waterMint: #B9F5D2;
     --color-darkBlue: #263B58;
     --color-lighterGray: #f6f6f6;
     --color-lightGray: #ddd;
@@ -11,17 +12,16 @@
     --color-limeGreen: #b9f5d2;
     --color-brightCyan: #46ece6;
     --color-lightCyan: #d4fbf9;
-    --color-darkCyan: #009099;
+    --color-darkCyan: #43a39f;
     --color-red: #c60f13;
     --color-darkRed: #5b2a2a;
 
     --background-color: var(--color-light);
     --color-accent: var(--color-brightCyan);
     --text-color: var(--color-dark);
-    --link-color: var(--color-darkCyan);
 
     /* Buttons. */
-    --button-primary-background: var(--color-limeGreen);
+    --button-primary-background: var(--color-waterMint);
     --button-primary-color: var(--color-darkBlue);
     --button-neutral-background: var(--color-lightGray);
     --button-neutral-color: var(--color-darkGray);

umap/templates/umap/map_list.html

@@ -12,14 +12,14 @@
             {% endfor %}
           </ul>
         {% endif %}
-        <h3>{{ map_inst.name }}</h3>
+        <h3><a href="{{ map_inst.get_absolute_url }}">{{ map_inst.name }}</a></h3>
         {% with author=map_inst.get_author %}
           {% if author %}
             <p>{% trans "by" %} <a href="{{ author.get_url }}">{{ author }}</a></p>
           {% endif %}
         {% endwith %}
       </div>
-      <a class="main" href="{{ map_inst.get_absolute_url }}">{% translate "See the map" %}</a>
+      <a class="button" href="{{ map_inst.get_absolute_url }}">{% translate "See the map" %}</a>
     </hgroup>
   </div>
 {% endfor %}

umap/templates/umap/search_bar.html

@@ -16,7 +16,7 @@
         <option value="{{ value }}" {% if request.GET.tags == value %}selected{% endif %}>{{ label }}</option>
         {% endfor %}
       </select>
-      <input type="submit" value="{% trans "Search" %}" />
+      <input type="submit" value="{% trans "Search" %}" class="neutral" />
     </form>
   </div>
 </div>

umap/tests/integration/test_browser.py

@@ -348,8 +348,9 @@ def test_should_redraw_list_on_feature_delete(live_server, openmap, page, bootst
     buttons = page.locator(".umap-browser .datalayer li .icon-delete")
     expect(buttons).to_have_count(3)
     buttons.first.click()
+    page.locator("dialog").get_by_role("button", name="OK").click()
     expect(buttons).to_have_count(2)
-    page.locator(".edit-undo").click()
+    page.get_by_role("button", name="Undo").click()
     expect(buttons).to_have_count(3)
 
 

umap/tests/integration/test_conditional_rules.py

@@ -261,7 +261,7 @@ def test_can_create_new_rule(live_server, page, openmap):
     page.get_by_title("AliceBlue").first.click()
     colors = getColors(markers)
     assert colors.count("rgb(240, 248, 255)") == 3
-    page.locator(".edit-undo").click()
+    page.get_by_role("button", name="Undo").click()
     colors = getColors(markers)
     assert colors.count("rgb(240, 248, 255)") == 0
 

umap/tests/integration/test_edit_datalayer.py

@@ -61,9 +61,10 @@ def test_cancel_deleting_datalayer_should_restore(
     expect(markers).to_have_count(1)
     page.get_by_role("button", name="Manage layers").click()
     page.locator(".panel.right").get_by_title("Delete layer").click()
+    page.get_by_role("button", name="OK").click()
     expect(markers).to_have_count(0)
     expect(page.get_by_text("test datalayer")).to_be_hidden()
-    page.locator(".edit-undo").click()
+    page.get_by_role("button", name="Undo").click()
     expect(markers).to_have_count(1)
     expect(page.locator(".umap-browser").get_by_text("test datalayer")).to_be_visible()
 
@@ -202,6 +203,7 @@ def test_deleting_datalayer_should_remove_from_browser_and_layers_list(
     expect(panel.get_by_text("test datalayer")).to_be_visible()
     expect(edit_panel.get_by_text("test datalayer")).to_be_visible()
     page.locator(".panel.right").get_by_title("Delete layer").click()
+    page.get_by_role("button", name="OK").click()
     expect(panel.get_by_text("test datalayer")).to_be_hidden()
     expect(edit_panel.get_by_text("test datalayer")).to_be_hidden()
 
@@ -215,6 +217,7 @@ def test_deleting_datalayer_should_remove_from_caption(
     page.get_by_role("button", name="Manage layers").click()
     expect(panel.get_by_text("test datalayer")).to_be_visible()
     page.locator(".panel.right").get_by_title("Delete layer").click()
+    page.get_by_role("button", name="OK").click()
     expect(panel.get_by_text("test datalayer")).to_be_hidden()
 
 

umap/tests/integration/test_edit_polygon.py

@@ -117,7 +117,7 @@ def test_should_reset_style_on_cancel(live_server, openmap, page, bootstrap):
     expect(page.locator(".leaflet-overlay-pane path[fill='GoldenRod']")).to_have_count(
         1
     )
-    page.locator(".edit-undo").click()
+    page.get_by_role("button", name="Undo").click()
     expect(page.locator(".leaflet-overlay-pane path[fill='DarkBlue']")).to_have_count(1)
 
 

umap/tests/integration/test_owned_map.py

@@ -241,6 +241,7 @@ def test_can_delete_datalayer(live_server, map, login, datalayer):
     expect(markers).to_have_count(1)
     page.get_by_role("button", name="Manage layers").click()
     page.locator(".panel.right").get_by_title("Delete layer").click()
+    page.get_by_role("button", name="OK").click()
     with page.expect_response(re.compile(r".*/datalayer/delete/.*")):
         page.get_by_role("button", name="Save").click()
     expect(markers).to_have_count(0)

umap/tests/integration/test_save.py

@@ -16,7 +16,7 @@ def test_reseting_map_would_remove_from_save_queue(
     page.on("request", register_request)
     page.locator('input[name="name"]').click()
     page.locator('input[name="name"]').fill("new name")
-    page.locator(".edit-undo").click()
+    page.get_by_role("button", name="Undo").click()
     page.wait_for_timeout(500)
     page.get_by_role("button", name="Manage layers").click()
     page.get_by_role("button", name="Edit", exact=True).click()

umap/tests/integration/test_websocket_sync.py

@@ -86,6 +86,7 @@ def test_websocket_connection_can_sync_markers(new_page, asgi_live_server, tilel
     # Delete a marker from peer A and check it's been deleted on peer B
     a_first_marker.click(button="right")
     peerA.get_by_role("button", name="Delete this feature").click()
+    peerA.locator("dialog").get_by_role("button", name="OK").click()
     expect(a_marker_pane).to_have_count(1)
     expect(b_marker_pane).to_have_count(1)
 
@@ -165,6 +166,7 @@ def test_websocket_connection_can_sync_polygons(context, asgi_live_server, tilel
     # Delete a polygon from peer A and check it's been deleted on peer B
     a_polygon.click(button="right")
     peerA.get_by_role("button", name="Delete this feature").click()
+    peerA.locator("dialog").get_by_role("button", name="OK").click()
     expect(a_polygons).to_have_count(0)
     expect(b_polygons).to_have_count(0)
 
@@ -483,6 +485,7 @@ def test_should_sync_datalayers_delete(new_page, asgi_live_server, tilelayer):
 
     # Delete "datalayer 2" in peerA
     peerA.locator(".datalayer").get_by_role("button", name="Delete layer").first.click()
+    peerA.get_by_role("button", name="OK").click()
     expect(peerA.locator(".panel").get_by_text("datalayer 2")).to_be_hidden()
     expect(peerB.locator(".panel").get_by_text("datalayer 2")).to_be_hidden()
 
@@ -683,7 +686,7 @@ def test_should_sync_datalayer_clear(
     expect(peerB.locator(".leaflet-marker-icon")).to_have_count(0)
 
     # Undo in peer A
-    peerA.locator(".edit-undo").click()
+    peerA.get_by_role("button", name="Undo").click()
     expect(peerA.locator(".leaflet-marker-icon")).to_have_count(1)
     expect(peerB.locator(".leaflet-marker-icon")).to_have_count(1)