From 759df9acfccd13cbf2c9f4d3ab64f99e2453be97 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20P=C3=A9ters?= <fpeters@entrouvert.com>
Date: Wed, 20 Oct 2021 21:55:38 +0200
Subject: [PATCH] workflows: include disabled handle (for layout) on readonly
 workflows (#58027)

---
 wcs/admin/workflows.py                            | 8 +++++---
 wcs/qommon/static/css/dc2/admin.scss              | 5 +++++
 wcs/templates/wcs/backoffice/workflow-status.html | 2 +-
 3 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/wcs/admin/workflows.py b/wcs/admin/workflows.py
index b84bba386..868b5f5b5 100644
--- a/wcs/admin/workflows.py
+++ b/wcs/admin/workflows.py
@@ -1262,7 +1262,7 @@ class GlobalActionPage(WorkflowStatusPage):
             r += htmltext('<p>%s</p>') % _('There are not yet any items in this action.')
         else:
             if self.workflow.is_readonly():
-                r += htmltext('<ul id="items-list" class="biglist">')
+                r += htmltext('<ul id="items-list" class="biglist readonly">')
             else:
                 r += htmltext('<p class="items">')
                 r += _('Use drag and drop with the handles to reorder items.')
@@ -1286,10 +1286,12 @@ class GlobalActionPage(WorkflowStatusPage):
             r += htmltext('</ul>')
         r += htmltext('</div>')  # bo-block
 
+        sortable = 'sortable readonly' if self.workflow.is_readonly() else 'sortable'
         r += htmltext('<div class="bo-block">')
         r += htmltext('<h2>%s</h2>') % _('Triggers')
-        r += htmltext(
-            '<ul id="items-list" class="biglist sortable" data-order-function="update_triggers_order">'
+        r += (
+            htmltext('<ul id="items-list" class="biglist %s" data-order-function="update_triggers_order">')
+            % sortable
         )
         for trigger in self.action.triggers:
             r += htmltext('<li class="biglistitem" id="trigId_%s">') % trigger.id
diff --git a/wcs/qommon/static/css/dc2/admin.scss b/wcs/qommon/static/css/dc2/admin.scss
index 8b84f739e..96b097eee 100644
--- a/wcs/qommon/static/css/dc2/admin.scss
+++ b/wcs/qommon/static/css/dc2/admin.scss
@@ -50,6 +50,11 @@ ul.biglist span.handle,
 	text-align: left;
 }
 
+ul.biglist.readonly span.handle {
+	opacity: 0.5;
+	cursor: not-allowed;
+}
+
 ul.biglist li,
 li.biglistitem {
 	list-style-type: none;
diff --git a/wcs/templates/wcs/backoffice/workflow-status.html b/wcs/templates/wcs/backoffice/workflow-status.html
index c22749c38..2f1c4971f 100644
--- a/wcs/templates/wcs/backoffice/workflow-status.html
+++ b/wcs/templates/wcs/backoffice/workflow-status.html
@@ -23,7 +23,7 @@
 {% spaceless %}
 <div class="bo-block">
   {% if workflow.is_readonly %}
-  <ul id="items-list" class="biglist">
+  <ul id="items-list" class="biglist sortable readonly">
   {% else %}
   <p class="hint">{% trans "Use drag and drop with the handles to reorder items." %}</p>
   <ul id="items-list" class="biglist sortable">