From 732839d5862eae3733130868ae78427cb455a669 Mon Sep 17 00:00:00 2001 From: Eugene Burmakin Date: Tue, 14 Oct 2025 13:47:41 +0200 Subject: [PATCH] Update icons on the map --- CHANGELOG.md | 1 + .../svg/icons/lucide/outline/square-dashed-mouse-pointer.svg | 1 + app/javascript/controllers/add_visit_controller.js | 4 ++-- app/javascript/maps/visits.js | 2 +- app/models/user.rb | 2 +- 5 files changed, 6 insertions(+), 4 deletions(-) create mode 100644 app/assets/svg/icons/lucide/outline/square-dashed-mouse-pointer.svg diff --git a/CHANGELOG.md b/CHANGELOG.md index 41b47751..1cb8e141 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -17,6 +17,7 @@ In this release we're introducing family features that allow users to create fam ## Fixed - Sign out button works again. #1844 +- Fixed user deletion bug where user could not be deleted due to counter cache on points. ## Changed diff --git a/app/assets/svg/icons/lucide/outline/square-dashed-mouse-pointer.svg b/app/assets/svg/icons/lucide/outline/square-dashed-mouse-pointer.svg new file mode 100644 index 00000000..a2ffde85 --- /dev/null +++ b/app/assets/svg/icons/lucide/outline/square-dashed-mouse-pointer.svg @@ -0,0 +1 @@ + diff --git a/app/javascript/controllers/add_visit_controller.js b/app/javascript/controllers/add_visit_controller.js index 672b0629..612d7a85 100644 --- a/app/javascript/controllers/add_visit_controller.js +++ b/app/javascript/controllers/add_visit_controller.js @@ -75,7 +75,7 @@ export default class extends Controller { const AddVisitControl = L.Control.extend({ onAdd: (map) => { const button = L.DomUtil.create('button', 'leaflet-control-button add-visit-button'); - button.innerHTML = '➕'; + button.innerHTML = ''; button.title = 'Add a visit'; // Style the button with theme-aware styling @@ -138,7 +138,7 @@ export default class extends Controller { // Reset button style with theme-aware styling applyThemeToButton(button, this.userThemeValue || 'dark'); - button.innerHTML = '➕'; + button.innerHTML = ''; // Reset cursor this.map.getContainer().style.cursor = ''; diff --git a/app/javascript/maps/visits.js b/app/javascript/maps/visits.js index 4a1bdf35..f22a94e5 100644 --- a/app/javascript/maps/visits.js +++ b/app/javascript/maps/visits.js @@ -104,7 +104,7 @@ export class VisitsManager { const SelectionControl = L.Control.extend({ onAdd: (map) => { const button = L.DomUtil.create('button', 'leaflet-bar leaflet-control leaflet-control-custom'); - button.innerHTML = '⚓️'; + button.innerHTML = ''; button.title = 'Select Area'; button.id = 'selection-tool-button'; // Style the button with theme-aware styling diff --git a/app/models/user.rb b/app/models/user.rb index c5d5d337..71269d64 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -5,7 +5,7 @@ class User < ApplicationRecord # rubocop:disable Metrics/ClassLength devise :database_authenticatable, :registerable, :recoverable, :rememberable, :validatable, :trackable - has_many :points, dependent: :destroy, counter_cache: true + has_many :points, dependent: :destroy has_many :imports, dependent: :destroy has_many :stats, dependent: :destroy has_many :exports, dependent: :destroy