From 11c278038352623d982c6695aef73c5d80144ca7 Mon Sep 17 00:00:00 2001
From: Sebastian Almberg <83243306+Sebbeben@users.noreply.github.com>
Date: Wed, 11 Mar 2026 16:29:35 +0100
Subject: [PATCH] Highlight top search result and remove skip reason prompt

- Highlight the recommended/top priority result row with table-success class
- Add "Top" badge to the recommended Quick Apply button
- Use outline style for non-top Quick Apply buttons to differentiate
- Remove the annoying "reason for skipping" prompt popup
---
 assets/controllers/bulk_import_controller.js         | 6 ++----
 templates/info_providers/bulk_import/step2.html.twig | 9 ++++++---
 translations/messages.en.xlf                         | 6 ++++++
 3 files changed, 14 insertions(+), 7 deletions(-)

diff --git a/assets/controllers/bulk_import_controller.js b/assets/controllers/bulk_import_controller.js
index c57f4c9b..a04ff13e 100644
--- a/assets/controllers/bulk_import_controller.js
+++ b/assets/controllers/bulk_import_controller.js
@@ -121,13 +121,11 @@ export default class extends Controller {
 
     async markSkipped(event) {
         const partId = event.currentTarget.dataset.partId
-        const reason = prompt('Reason for skipping (optional):') || ''
-        
+
         try {
             const url = this.markSkippedUrlValue.replace('__PART_ID__', partId)
             const data = await this.fetchWithErrorHandling(url, {
-                method: 'POST',
-                body: JSON.stringify({ reason })
+                method: 'POST'
             })
             
             if (data.success) {
diff --git a/templates/info_providers/bulk_import/step2.html.twig b/templates/info_providers/bulk_import/step2.html.twig
index 30b0d386..62988282 100644
--- a/templates/info_providers/bulk_import/step2.html.twig
+++ b/templates/info_providers/bulk_import/step2.html.twig
@@ -208,11 +208,13 @@
                                 </tr>
                             </thead>
                             <tbody>
-                                {% for result in part_result.searchResults %}
+                                {% set sortedResults = part_result.resultsSortedByPriority %}
+                                {% for result in sortedResults %}
                                     {# @var result \App\Services\InfoProviderSystem\DTOs\BulkSearchPartResultDTO #}
                                     {% set dto = result.searchResult %}
                                     {% set localPart = result.localPart %}
-                                    <tr>
+                                    {% set isTopResult = loop.first %}
+                                    <tr{% if isTopResult and not isCompleted %} class="table-success"{% endif %}>
                                         <td>
                                             <img src="{{ dto.preview_image_url }}" data-thumbnail="{{ dto.preview_image_url }}"
                                                  class="hoverpic" style="max-width: 35px;" {{ stimulus_controller('elements/hoverpic') }}>
@@ -242,13 +244,14 @@
                                         <td>
                                             <div class="btn-group-vertical btn-group-sm" role="group">
                                                 {% if not isCompleted %}
-                                                    <button type="button" class="btn btn-success"
+                                                    <button type="button" class="btn btn-success{% if not isTopResult %} btn-outline-success{% endif %}"
                                                             data-action="click->bulk-import#quickApply"
                                                             data-part-id="{{ part.id }}"
                                                             data-provider-key="{{ dto.provider_key }}"
                                                             data-provider-id="{{ dto.provider_id }}"
                                                             title="{% trans %}info_providers.bulk_import.quick_apply.tooltip{% endtrans %}">
                                                         <i class="fas fa-bolt"></i> {% trans %}info_providers.bulk_import.quick_apply{% endtrans %}
+                                                        {% if isTopResult %}<span class="badge bg-light text-success ms-1">{% trans %}info_providers.bulk_import.recommended{% endtrans %}</span>{% endif %}
                                                     </button>
                                                 {% endif %}
                                                 {% set updateHref = path('info_providers_update_part',
diff --git a/translations/messages.en.xlf b/translations/messages.en.xlf
index 1c821362..7bb98393 100644
--- a/translations/messages.en.xlf
+++ b/translations/messages.en.xlf
@@ -11127,6 +11127,12 @@ Please note, that you can not impersonate a disabled user. If you try you will g
         <target>All parts have been processed. You can review the results below or navigate back to the parts list.</target>
       </segment>
     </unit>
+    <unit id="recommended_badge" name="info_providers.bulk_import.recommended">
+      <segment state="translated">
+        <source>info_providers.bulk_import.recommended</source>
+        <target>Top</target>
+      </segment>
+    </unit>
     <unit id="quick_apply_btn" name="info_providers.bulk_import.quick_apply">
       <segment state="translated">
         <source>info_providers.bulk_import.quick_apply</source>