From 0db5d26c6afcf4051341097acc84baef7e2d11a1 Mon Sep 17 00:00:00 2001
From: Frank Elsinga <frank@elsinga.de>
Date: Sun, 25 Jun 2023 15:28:11 +0200
Subject: [PATCH] removed all traces of the coordinate-picker to make space for
 a better system

---
 .../feedback/src/proposed_edits/tmp_repo.rs   |   2 +-
 webclient/src/components/AppFeedbackModal.vue |  36 +----
 .../src/components/DetailsFeedbackButton.vue  |  97 +------------
 .../src/components/DetailsInfoSection.vue     |   6 -
 .../src/components/DetailsInteractiveMap.vue  | 107 --------------
 webclient/src/locales/de.yaml                 |  24 ----
 webclient/src/locales/en.yaml                 |  24 ----
 webclient/src/pages/view/[id].vue             | 133 +-----------------
 webclient/src/stores/details.ts               |  10 --
 9 files changed, 9 insertions(+), 430 deletions(-)

diff --git a/server/feedback/src/proposed_edits/tmp_repo.rs b/server/feedback/src/proposed_edits/tmp_repo.rs
index 7f120acc3..423cf1c5b 100644
--- a/server/feedback/src/proposed_edits/tmp_repo.rs
+++ b/server/feedback/src/proposed_edits/tmp_repo.rs
@@ -64,7 +64,7 @@ impl TempRepo {
         let out = Command::new("git")
             .current_dir(&self.dir)
             .arg("commit")
-            .arg("--allow-empty") // TODO: remove once the need for empty commits is gone (images+coodinates)
+            .arg("--allow-empty") // TODO: remove once the need for empty commits is gone (images+coordinates)
             .arg("--all") // run git add . before commit
             .arg("-m")
             .arg(title)
diff --git a/webclient/src/components/AppFeedbackModal.vue b/webclient/src/components/AppFeedbackModal.vue
index cbd2d9470..cf0d0001b 100644
--- a/webclient/src/components/AppFeedbackModal.vue
+++ b/webclient/src/components/AppFeedbackModal.vue
@@ -109,11 +109,6 @@ function sendForm() {
         <div class="content">
           <div id="feedback-error">{{ error.message }}</div>
           <div class="form-group">
-            <div id="feedback-coordinate-picker-helptext" class="d-none toast toast-primary">
-              {{ $t("feedback.coordinatepicker.helptext.enter_serveral") }}<br />
-              {{ $t("feedback.coordinatepicker.helptext.saved_for_12h") }}<br />
-              {{ $t("feedback.coordinatepicker.helptext.limitation_to_coordinates") }}
-            </div>
             <label class="form-label" for="feedback-subject"> {{ $t("feedback.subject") }}</label>
             <div class="input-group">
               <select
@@ -139,18 +134,9 @@ function sendForm() {
           </div>
 
           <div class="form-group">
-            <div>
-              <label class="form-label" for="feedback-body">
-                {{ $t("feedback.message") }}
-              </label>
-              <button
-                id="feedback-coordinate-picker"
-                v-if="global.feedback.category === 'entry'"
-                class="btn btn-sm btn-link"
-              >
-                {{ $t("feedback.coordinatepicker.title") }}
-              </button>
-            </div>
+            <label class="form-label" for="feedback-body">
+              {{ $t("feedback.message") }}
+            </label>
             <textarea
               class="form-input"
               id="feedback-body"
@@ -173,13 +159,6 @@ function sendForm() {
             </p>
           </div>
 
-          <!-- only visible if called through a view, because then the context of the calling building is availible -->
-          <div>
-            <button id="feedback-coordinate-picker" class="btn btn-sm d-none">
-              {{ $t("feedback.coordinatepicker.title") }}
-            </button>
-          </div>
-
           <div class="form-group">
             <label class="form-checkbox">
               <input type="checkbox" id="feedback-privacy" v-model="privacyChecked" />
@@ -305,14 +284,5 @@ function sendForm() {
   #feedback-body {
     min-width: 100%;
   }
-
-  #feedback-coordinate-picker {
-    float: right;
-    margin-top: 0.5em;
-  }
-
-  #feedback-coordinate-picker-helptext {
-    font-size: 14px;
-  }
 }
 </style>
diff --git a/webclient/src/components/DetailsFeedbackButton.vue b/webclient/src/components/DetailsFeedbackButton.vue
index d429eb36c..d9e74d08b 100644
--- a/webclient/src/components/DetailsFeedbackButton.vue
+++ b/webclient/src/components/DetailsFeedbackButton.vue
@@ -1,105 +1,16 @@
 <script setup lang="ts">
-import { getLocalStorageWithExpiry } from "@/composables/storage";
-import { useDetailsStore } from "@/stores/details";
 import { useGlobalStore } from "@/stores/global";
-import { useI18n } from "vue-i18n";
-import type { Coord } from "@/stores/global";
+import { useRoute } from "vue-router";
 
-const { t } = useI18n({ inheritLocale: true, useScope: "global" });
-const state = useDetailsStore();
-type CurrentEdits = { [index: string]: Coord };
-function _getFeedbackSubject(currentEdits: CurrentEdits) {
-  if (Object.keys(currentEdits).length > 1) {
-    return `[${state.data?.id} et.al.]: ` + t("feedback.coordinatepicker.edit_coordinates_subject");
-  }
-
-  const subjectPrefix = `[${state.data?.id}]: `;
-  const subjectMsg =
-    Object.keys(currentEdits).length === 0 ? "" : t("feedback.coordinatepicker.edit_coordinate_subject");
-
-  // The subject backup is only loaded (and supported) when a single
-  // entry is being edited
-  if (
-    state.coord_picker.subject_backup &&
-    state.coord_picker.backup_id === state.data?.id &&
-    state.coord_picker.subject_backup !== subjectPrefix
-  ) {
-    const backup = state.coord_picker.subject_backup;
-    state.coord_picker.subject_backup = null;
-    return backup;
-  }
-  return subjectPrefix + subjectMsg;
-}
-function _getFeedbackBody(currentEdits: CurrentEdits) {
-  if (!state.data) {
-    return "Data has not been loaded yet but Feedback is open. This is a bug. Please describe how you got here.";
-  }
-  // Look up whether there is a backup of the body and extract the section
-  // that is not the coordinate
-  let actionMsg = "";
-  if (state.coord_picker.body_backup && state.coord_picker.backup_id === state.data.id) {
-    const parts = state.coord_picker.body_backup.split("\n```");
-    if (parts.length === 1) {
-      actionMsg = parts[0];
-    } else {
-      actionMsg = parts[0] + parts[1].split("```").slice(1).join("\n");
-    }
-
-    state.coord_picker.body_backup = null;
-  }
-
-  if (Object.keys(currentEdits).length === 0) {
-    // For no edits, don't show a badly formatted message
-    // (This is "" if there was no backup)
-    return actionMsg;
-  }
-
-  const defaultActionMsg =
-    state.data.coords.accuracy === "building"
-      ? t("feedback.coordinatepicker.add_coordinate")
-      : t("feedback.coordinatepicker.correct_coordinate");
-  actionMsg = actionMsg || defaultActionMsg;
-
-  if (Object.keys(currentEdits).length > 1) {
-    // The body backup is discarded if more than a single entry
-    // is being edited (because then it is not supported).
-    actionMsg = t("feedback.coordinatepicker.edit_multiple_coordinates");
-  }
-
-  let editStr = "";
-  Object.entries(currentEdits).forEach(([key, value]) => {
-    editStr += `"${key}": { lat: ${value.coords.lat}, lon: ${value.coords.lon} }\n`;
-  });
-
-  return `${actionMsg}\n\`\`\`yaml\n${editStr}\`\`\``;
-}
-
-defineExpose({
-  openFeedbackForm,
-});
-function openFeedbackForm(addLocationPicker: EventListener | null = null) {
-  // The feedback form is opened. This may be prefilled with previously corrected coordinates.
-  // Maybe get the old coordinates from localstorage
-  const currentEdits = getLocalStorageWithExpiry<CurrentEdits>("feedback-coords", {});
-  const body = _getFeedbackBody(currentEdits);
-  const subject = _getFeedbackSubject(currentEdits);
-
-  if (addLocationPicker !== null) {
-    window.setTimeout(
-      () => document.getElementById("feedback-coordinate-picker")?.addEventListener("click", addLocationPicker),
-      100
-    );
-  }
-
-  useGlobalStore().openFeedback("entry", subject, body);
-}
+const route = useRoute();
+const global = useGlobalStore();
 </script>
 
 <template>
   <button
     class="btn btn-link btn-action btn-sm"
     :title="$t('view_view.header.feedback')"
-    @click="openFeedbackForm()"
+    @click="global.openFeedback('entry', `[${route.params.id}]: `)"
     data-cy="open-feedback-details"
   >
     <i class="icon icon-flag" />
diff --git a/webclient/src/components/DetailsInfoSection.vue b/webclient/src/components/DetailsInfoSection.vue
index 0467c795a..c1dfb4feb 100644
--- a/webclient/src/components/DetailsInfoSection.vue
+++ b/webclient/src/components/DetailsInfoSection.vue
@@ -37,12 +37,6 @@ const state = useDetailsStore();
         <DetailsPropertyTable />
         <div class="toast toast-warning" v-if="state.data?.coords.accuracy === 'building'">
           {{ $t("view_view.msg.inaccurate_only_building.primary_msg") }}<br />
-          <i>
-            {{ $t("view_view.msg.inaccurate_only_building.help_others_and") }}
-            <button class="btn btn-sm" @click="addLocationPicker">
-              {{ $t("view_view.msg.inaccurate_only_building.btn") }}
-            </button>
-          </i>
         </div>
         <div
           class="toast toast-warning"
diff --git a/webclient/src/components/DetailsInteractiveMap.vue b/webclient/src/components/DetailsInteractiveMap.vue
index d2d35066e..add3c5583 100644
--- a/webclient/src/components/DetailsInteractiveMap.vue
+++ b/webclient/src/components/DetailsInteractiveMap.vue
@@ -1,10 +1,8 @@
 <script setup lang="ts">
-import { getLocalStorageWithExpiry, setLocalStorageWithExpiry } from "@/composables/storage";
 import type { BackgroundLayerSpecification, Coordinates, ImageSource } from "maplibre-gl";
 import { Map, Marker } from "maplibre-gl";
 import { AttributionControl, FullscreenControl, GeolocateControl, NavigationControl } from "maplibre-gl";
 import { selectedMap, useDetailsStore } from "@/stores/details";
-import type { Coord } from "@/stores/global";
 import { useGlobalStore } from "@/stores/global";
 import { nextTick, ref } from "vue";
 import { FloorControl } from "@/modules/FloorControl";
@@ -12,98 +10,15 @@ import { webglSupport } from "@/composables/webglSupport";
 
 const map = ref<Map | undefined>(undefined);
 const marker = ref<Marker | undefined>(undefined);
-const marker2 = ref<Marker | null>(null);
 const floorControl = ref<FloorControl>(new FloorControl());
 const state = useDetailsStore();
 const global = useGlobalStore();
 
-// The coordinate picker keeps backups of the subject and body
-// in case someone writes a text and then after that clicks
-// the set coordinate button in the feedback form.
-// If we no backup has been made then, this would be lost after clicking confirm there.
-const coord_picker = ref({
-  backup_id: null as string | null,
-  subject_backup: null as string | null,
-  body_backup: null as string | null,
-  force_reopen: false,
-});
 const initialLoaded = ref(false);
 
-const emit = defineEmits<{
-  (e: "openFeedbackForm", callback: EventListener): void;
-}>();
-function confirmLocationPicker() {
-  // add the current edits to the feedback
-  const currentEdits = getLocalStorageWithExpiry<{ [index: string]: Coord }>("feedback-coords", {});
-  const location = marker2.value?.getLngLat();
-  currentEdits[state.data?.id || "undefined"] = {
-    coords: { lat: location?.lat, lon: location?.lng },
-  };
-  // save to local storage with ttl of 12h (garbage-collected on next read)
-  setLocalStorageWithExpiry("feedback-coords", currentEdits, 12);
-
-  marker2.value?.remove();
-  marker2.value = null;
-
-  // A feedback form is only opened when this is the only (and therefore
-  // first coordinate). If there are more coordinates we can assume
-  // someone is doing batch edits. They can then use the send button in
-  // the coordinate counter at the top of the page.
-  if (Object.keys(currentEdits).length === 1 || state.coord_picker.force_reopen) {
-    state.coord_picker.force_reopen = false;
-    emit("openFeedbackForm", () => addLocationPicker());
-  }
-
-  // The helptext (which says thet you can edit multiple coordinates in bulk)
-  // is also only shown if there is one edit.
-  if (Object.keys(currentEdits).length === 1) {
-    document.getElementById("feedback-coordinate-picker-helptext")?.classList.remove("d-none");
-  }
-}
-function cancelLocationPicker() {
-  marker2.value?.remove();
-  marker2.value = null;
-
-  if (state.coord_picker.force_reopen) {
-    state.coord_picker.force_reopen = false;
-    emit("openFeedbackForm", () => addLocationPicker());
-  }
-}
 defineExpose({
-  addLocationPicker,
   loadInteractiveMap,
 });
-function addLocationPicker() {
-  // If this is called from the feedback form using the edit coordinate
-  // button, we temporarily save the current subject and body, so it is
-  // not lost when being reopened
-  if (global.feedback.open) {
-    coord_picker.value.backup_id = state.data?.id || "undefined";
-    coord_picker.value.subject_backup = global.feedback.subject;
-    coord_picker.value.body_backup = global.feedback.body;
-    coord_picker.value.force_reopen = true; // reopen after confirm
-
-    global.temporarilyCloseFeedback();
-  }
-
-  state.map.selected = selectedMap.interactive;
-
-  // Verify that there isn't already a marker (could happen if you click 'assign
-  // a location' multiple times from the 'missing accurate location' toast)
-  if (marker2.value === null) {
-    // Coordinates are either taken from the entry, or if there are already
-    // some in the localStorage use them
-    const currentEdits = getLocalStorageWithExpiry<{ [index: string]: Coord }>("feedback-coords", {});
-
-    const { coords } = currentEdits[state.data?.id || "undefined"] || state.data;
-    marker2.value = new Marker({
-      draggable: true,
-      color: "#ff0000",
-    });
-    if (coords.lat !== undefined && coords.lon !== undefined)
-      marker2.value.setLngLat([coords.lon, coords.lat]).addTo(map.value as Map);
-  }
-}
 
 function loadInteractiveMap(fromUi = false) {
   if (!webglSupport) return;
@@ -346,22 +261,6 @@ function setOverlayImage(imgUrl: string | null, coords: Coordinates | undefined)
 </script>
 
 <template>
-  <div class="toast toast-primary mb-2 location-picker" v-if="marker2">
-    <div class="columns">
-      <div class="column col col-sm-12">
-        {{ $t("view_view.msg.correct_location.msg") }}
-      </div>
-      <div class="column col-auto col-sm-12 btns">
-        <button class="btn btn-sm" @click="cancelLocationPicker">
-          {{ $t("view_view.msg.correct_location.btn-cancel") }}
-        </button>
-        <button class="btn btn-sm" @click="confirmLocationPicker">
-          <i class="icon icon-check" />
-          {{ $t("view_view.msg.correct_location.btn-done") }}
-        </button>
-      </div>
-    </div>
-  </div>
   <div
     id="interactive-map-container"
     :class="{ 'd-none': state.map.selected !== selectedMap.interactive, errormessage: !webglSupport }"
@@ -397,12 +296,6 @@ function setOverlayImage(imgUrl: string | null, coords: Coordinates | undefined)
   // This does not change anything (except using px instead of rem),
   // but ensures that roomfinder position calculations are predictable.
   padding: 0 8px;
-
-  // The marker2 (draggable)
-  .maplibregl-marker + .maplibregl-marker {
-    animation: fade-in 0.1s linear 0.05s;
-    animation-fill-mode: both;
-  }
 }
 
 .toast.location-picker {
diff --git a/webclient/src/locales/de.yaml b/webclient/src/locales/de.yaml
index 3d9a431d9..a7bd089ce 100644
--- a/webclient/src/locales/de.yaml
+++ b/webclient/src/locales/de.yaml
@@ -11,17 +11,6 @@ core_js:
 feedback:
   cancel: Abbrechen
   category: Feedback-Kategorie
-  coordinatepicker:
-    add_coordinate: 'Hallo, ich möchte diese Koordinate zum Roomfinder hinzufügen:'
-    correct_coordinate: 'Hallo, ich möchte diese Koordinate im Roomfinder korrigieren:'
-    edit_coordinate_subject: Koordinate bearbeiten
-    edit_coordinates_subject: Koordinaten bearbeiten
-    edit_multiple_coordinates: 'Hallo, ich möchte diese Koordinaten beim Roomfinder editieren:'
-    helptext:
-      enter_serveral: Falls du mehrere Koordinaten gleichzeitig korrigieren willst, kannst du dieses Fenster schließen und weitere Koordinaten editieren.
-      limitation_to_coordinates: Achtung, das gilt nur für Koordinaten!
-      saved_for_12h: Wir speichern deine Veränderungen für 12 Stunden bei dir lokal.
-    title: Koordinate auswählen
   delete: Das zugehörige GitHub Issue löschen, sobald es gelöst wurde.
   error:
     too_many_requests: Feedback senden ist aktuell nicht möglich aufgrund von rate-limiting. Bitte versuche es später nochmal oder schreibe eine Mail.
@@ -184,20 +173,7 @@ view_view:
   meta:
     details_for: Details für
   msg:
-    coordinate-counter:
-      delete: Löschen
-      delete-confirm: Wirklich löschen?
-      info: Änderungen werden lokal \n für 12h gespeichert
-      msg-1: Aktuell
-      msg-2: ausstehende Koordinatenänderung|ausstehende Koordinatenänderungen
-      send: Senden
-    correct_location:
-      btn-cancel: Abbrechen
-      btn-done: Fertig
-      msg: Um die richtige Position zu setzen, ziehe den roten Marker über die Karte.
     inaccurate_only_building:
-      btn: Koordinate eintragen
-      help_others_and: Falls du sie herausfindest, kannst du die
       primary_msg: Die angezeigte Position zeigt nur die Position des Gebäude(teils). Die genaue Lage innerhalb des Gebäudes ist uns nicht bekannt.
     no_floor_overlay: Für den angezeigten Raum gibt es leider keine Indoor Karte.
   roomfinder_modal:
diff --git a/webclient/src/locales/en.yaml b/webclient/src/locales/en.yaml
index 3f35b145b..c27989235 100644
--- a/webclient/src/locales/en.yaml
+++ b/webclient/src/locales/en.yaml
@@ -11,17 +11,6 @@ core_js:
 feedback:
   cancel: Cancel
   category: Feedback category
-  coordinatepicker:
-    add_coordinate: 'Hello, I would like to add this coordinate to the roomfinder:'
-    correct_coordinate: 'Hello, I would like to correct this coordinate in the roomfinder:'
-    edit_coordinate_subject: Edit coordinate
-    edit_coordinates_subject: Edit coordinates
-    edit_multiple_coordinates: 'Hello, I would like to edit these coordinates in the roomfinder:'
-    helptext:
-      enter_serveral: If you want to correct several coordinates at the same time, you can close this window and edit more coordinates.
-      limitation_to_coordinates: Warning, that's only valid for coordinates!
-      saved_for_12h: We will save your changes locally for 12 hours.
-    title: Select coordinate
   delete: Delete this GitHub issue when resolved.
   error:
     too_many_requests: Sending feedback is currently not possible due to rate-limiting. Please try again in a while or send a mail.
@@ -184,20 +173,7 @@ view_view:
   meta:
     details_for: Details for
   msg:
-    coordinate-counter:
-      delete: Delete
-      delete-confirm: Really delete?
-      info: Changes are stored \n locally for 12 h
-      msg-1: Currently
-      msg-2: pending coordinate edit|pending coordinate edits
-      send: Send
-    correct_location:
-      btn-cancel: Cancel
-      btn-done: Done
-      msg: To set the location, drag the red Marker around.
     inaccurate_only_building:
-      btn: Assign a coordinate
-      help_others_and: If you find it out, you can help others and
       primary_msg: The displayed position only shows the position of the building(part). The exact position within the building is not known to us.
     no_floor_overlay: There is unfortunately no indoor map for the displayed room.
   roomfinder_modal:
diff --git a/webclient/src/pages/view/[id].vue b/webclient/src/pages/view/[id].vue
index 443b5dc9f..12415a8c4 100644
--- a/webclient/src/pages/view/[id].vue
+++ b/webclient/src/pages/view/[id].vue
@@ -55,11 +55,6 @@ watchEffect(() => {
 const state = useDetailsStore();
 const clipboardSource = computed(() => `https://nav.tum.de${route.fullPath}`);
 const { copy, copied, isSupported: clipboardIsSupported } = useClipboard({ source: clipboardSource });
-// Coordinate picker states
-const coord_counter = ref({
-  counter: null as number | null,
-  to_confirm_delete: false,
-});
 
 function genDescription(d: DetailsResponse) {
   const detailsFor = t("view_view.meta.details_for");
@@ -73,18 +68,6 @@ function genDescription(d: DetailsResponse) {
   return description;
 }
 // --- Loading components ---
-function deletePendingCoordinates() {
-  if (coord_counter.value.to_confirm_delete) {
-    removeLocalStorage("feedback-coords");
-    coord_counter.value.to_confirm_delete = false;
-    state.coord_picker.body_backup = null;
-    state.coord_picker.subject_backup = null;
-    state.coord_picker.backup_id = null;
-  } else {
-    coord_counter.value.to_confirm_delete = true;
-  }
-}
-
 function tryToLoadMap() {
   /**
    * Try to load the entry map (interactive or roomfinder). It requires the map container
@@ -106,19 +89,12 @@ const roomfinderMap = ref<InstanceType<typeof DetailsRoomfinderMap> | null>(null
 onMounted(() => {
   if (navigator.userAgent === "Rendertron") return;
 
-  // Update pending coordinate counter on localStorage changes
-  const updateCoordinateCounter = function () {
-    const coords = getLocalStorageWithExpiry<{ [index: string]: Coord }>("feedback-coords", {});
-    coord_counter.value.counter = Object.keys(coords).length;
-  };
-  window.addEventListener("storage", updateCoordinateCounter);
   window.addEventListener("resize", () => {
     if (state.map.selected === selectedMap.roomfinder) {
       roomfinderMap.value?.loadRoomfinderMap(state.map.roomfinder.selected_index);
       roomfinderMap.value?.loadRoomfinderModalMap();
     }
   });
-  updateCoordinateCounter();
 
   nextTick(() => {
     // Even though 'mounted' is called there is no guarantee apparently,
@@ -156,44 +132,6 @@ onMounted(() => {
       />
     </a>
 
-    <!-- Pending coordinates counter (if any) -->
-    <div class="panel coord-counter" v-if="coord_counter.counter">
-      <div class="panel-body columns">
-        <div class="column col col-sm-12 msg">
-          {{ $t("view_view.msg.coordinate-counter.msg-1") }}
-          <em>{{ coord_counter.counter }} </em>
-          <span>
-            {{ $t("view_view.msg.coordinate-counter.msg-2", coord_counter.counter) }}
-          </span>
-          <button
-            class="btn btn-action btn-sm btn-link tooltip tooltip-left"
-            :data-tooltip="$t('view_view.msg.coordinate-counter.info')"
-          >
-            &#x1f6c8;
-          </button>
-        </div>
-        <div class="column col-auto col-sm-12 btns">
-          <button
-            class="btn btn-link btn-sm delete"
-            :class="{ 'to-confirm': coord_counter.to_confirm_delete }"
-            @click="deletePendingCoordinates"
-          >
-            <i class="icon icon-cross" />
-            <span class="default">
-              {{ $t("view_view.msg.coordinate-counter.delete") }}
-            </span>
-            <span class="confirm">
-              {{ $t("view_view.msg.coordinate-counter.delete-confirm") }}
-            </span>
-          </button>
-          <button class="btn btn-primary btn-sm" @click="feedbackButton?.openFeedbackForm()">
-            <i class="icon icon-check" />
-            {{ $t("view_view.msg.coordinate-counter.send") }}
-          </button>
-        </div>
-      </div>
-    </div>
-
     <!-- Breadcrumbs -->
     <ol class="breadcrumb" vocab="https://schema.org/" typeof="BreadcrumbList">
       <li
@@ -294,15 +232,6 @@ onMounted(() => {
       <!-- Map container -->
       <div class="column col-7 col-md-12" id="map-container">
         <div class="show-sm">
-          <div class="toast toast-warning" v-if="state.data.coords.accuracy === 'building'">
-            {{ $t("view_view.msg.inaccurate_only_building.primary_msg") }}<br />
-            <i>
-              {{ $t("view_view.msg.inaccurate_only_building.help_others_and") }}
-              <button class="btn btn-sm" @click="addLocationPicker">
-                {{ $t("view_view.msg.inaccurate_only_building.btn") }}
-              </button>
-            </i>
-          </div>
           <div
             class="toast toast-warning"
             v-if="state.data?.type === 'room' && state.data?.maps?.overlays?.default === null"
@@ -314,10 +243,7 @@ onMounted(() => {
           </div>
         </div>
 
-        <DetailsInteractiveMap
-          ref="interactiveMap"
-          @open-feedback-form="(addLocationPicker) => feedbackButton?.openFeedbackForm(addLocationPicker)"
-        />
+        <DetailsInteractiveMap ref="interactiveMap"/>
         <DetailsRoomfinderMap ref="roomfinderMap" />
         <div class="btn-group btn-group-block">
           <button
@@ -460,63 +386,6 @@ onMounted(() => {
     }
   }
 
-  /* --- Pending coordinates counter --- */
-  .coord-counter {
-    margin: 8px 0;
-    box-shadow: $card-shadow;
-    border: 1px solid $card-border;
-    border-radius: 4px;
-    background: $card-highlighted-bg;
-
-    & .panel-body {
-      overflow-y: visible;
-
-      & .msg {
-        margin: 15px 0;
-
-        & em {
-          color: $theme-accent;
-          font-style: normal;
-          font-weight: bold;
-        }
-
-        & .btn {
-          height: 1.3rem;
-          width: 1.3rem;
-
-          &::after {
-            z-index: 2000;
-          }
-        }
-      }
-
-      & .btns {
-        margin: auto 0 12px;
-
-        .delete .default {
-          display: inline-block;
-        }
-
-        .delete .confirm {
-          display: none;
-        }
-
-        .delete.to-confirm {
-          animation: delay-btn 0.3s steps(1);
-          animation-fill-mode: both;
-        }
-
-        .delete.to-confirm .default {
-          display: none;
-        }
-
-        .delete.to-confirm .confirm {
-          display: inline-block;
-        }
-      }
-    }
-  }
-
   /* --- Sections general --- */
   section {
     margin-top: 40px;
diff --git a/webclient/src/stores/details.ts b/webclient/src/stores/details.ts
index d21a95e1b..df08ca8a6 100644
--- a/webclient/src/stores/details.ts
+++ b/webclient/src/stores/details.ts
@@ -32,16 +32,6 @@ export const useDetailsStore = defineStore({
         height: 300,
       },
     },
-    coord_picker: {
-      // The coordinate picker keeps backups of the subject and body
-      // in case someone writes a text and then after that clicks
-      // the set coordinate button in the feedback form. If we wouldn't
-      // make a backup, this would be lost after clicking confirm there.
-      backup_id: null as string | null,
-      subject_backup: null as string | null,
-      body_backup: null as string | null,
-      force_reopen: false,
-    },
   }),
   actions: {
     selectedRoomfinderMap: function (): RoomfinderMapEntry {