Fix a bug where the confirmation alert was shown more than once when deleting a point.

This commit is contained in:
Eugene Burmakin 2024-09-02 23:29:37 +02:00
parent f99725e2f3
commit 740d203647
3 changed files with 20 additions and 2 deletions

View file

@ -1,2 +1,8 @@
/log
/tmp
# We need directories for import and export files, but not the files themselves.
/public/exports/*
!/public/exports/.keep
/public/imports/*
!/public/imports/.keep

View file

@ -36,6 +36,10 @@ Example of a valid point in GeoJSON format:
- Default exporting format is now GeoJSON instead of Owntracks-like JSON. This will allow you to use the exported data in other applications that support GeoJSON format.
### Fixed
- Fixed a bug where the confirmation alert was shown more than once when deleting a point.
## [0.12.3] — 2024-09-02

View file

@ -145,8 +145,12 @@ export default class extends Controller {
`;
}
removeEventListeners() {
document.removeEventListener('click', this.handleDeleteClick);
}
addEventListeners() {
document.addEventListener('click', (event) => {
this.handleDeleteClick = (event) => {
if (event.target && event.target.classList.contains('delete-point')) {
event.preventDefault();
const pointId = event.target.getAttribute('data-id');
@ -155,7 +159,11 @@ export default class extends Controller {
this.deletePoint(pointId, this.apiKey);
}
}
});
};
// Ensure only one listener is attached by removing any existing ones first
this.removeEventListeners();
document.addEventListener('click', this.handleDeleteClick);
}
deletePoint(id, apiKey) {