code quality increased in js

Author: Dileepadari

administrator/components/com_workflow/layouts/toolbar/redo.php

@@ -18,8 +18,14 @@
 
 ?>
 <joomla-toolbar-button>
-    <button class="btn btn-info action-button" onclick="WorkflowGraph.Event.fire('onClickRedoWorkflow');" tabindex="0">
+    <button id="redo-workflow" class="btn btn-info action-button" tabindex="0">
         <span class="icon-redo icon-fw" aria-hidden="true"></span>
         <?php echo Text::_('COM_WORKFLOW_REDO'); ?>
     </button>
 </joomla-toolbar-button>
+
+<script>
+    document.getElementById('redo-workflow')?.addEventListener('click', () => {
+        WorkflowGraph.Event.fire('onClickRedoWorkflow');
+    });
+</script>

administrator/components/com_workflow/layouts/toolbar/undo.php

@@ -18,8 +18,16 @@
 
 ?>
 <joomla-toolbar-button>
-    <button class="btn btn-info action-button" onclick="WorkflowGraph.Event.fire('onClickUndoWorkflow');" tabindex="0">
+    <button id="undo-workflow" class="btn btn-info action-button" tabindex="0">
         <span class="icon-undo-2 icon-fw" aria-hidden="true"></span>
         <?php echo Text::_('COM_WORKFLOW_UNDO'); ?>
     </button>
 </joomla-toolbar-button>
+
+
+<script>
+    document.getElementById('undo-workflow')?.addEventListener('click', () => {
+        WorkflowGraph.Event.fire('onClickUndoWorkflow');
+    });
+</script>
+

administrator/components/com_workflow/resources/scripts/components/Titlebar.vue

@@ -11,37 +11,33 @@
         class="d-flex align-items-center flex-wrap mb-0"
         aria-label="workflow details"
       >
-        <div class="me-3 mb-1 d-flex">
-          <dt class="visually-hidden">
-            {{ sprintf('COM_WORKFLOW_GRAPH_STATUS', workflow.published ? 'COM_WORKFLOW_GRAPH_ENABLED' : 'COM_WORKFLOW_GRAPH_DISABLED') }}
-          </dt>
-          <dd>
-            <span
-              class="badge"
-              :class="workflow.published ? 'bg-success' : 'bg-warning'"
-              role="status"
-            >
-              {{ workflow.published ? translate('COM_WORKFLOW_GRAPH_ENABLED') : translate('COM_WORKFLOW_GRAPH_DISABLED') }}
-            </span>
-          </dd>
-        </div>
-        <div class="me-3 mb-1 d-flex">
-          <dt class="visually-hidden">
-            {{ sprintf('COM_WORKFLOW_GRAPH_STAGE_COUNT', stagesCount) }}
-          </dt>
-          <dd>
-            {{ stagesCount }} {{ stagesCount === 1 ? translate('COM_WORKFLOW_GRAPH_STAGE') : translate('COM_WORKFLOW_GRAPH_STAGES') }}
-          </dd>
-        </div>
-        <div class="me-3 mb-1 d-flex">
-          <dt class="visually-hidden">
-            {{ sprintf('COM_WORKFLOW_GRAPH_TRANSITION_COUNT', transitionsCount) }}
-          </dt>
-          <dd>
-            {{ transitionsCount }} {{ transitionsCount === 1 ? translate('COM_WORKFLOW_GRAPH_TRANSITION') :
-              translate('COM_WORKFLOW_GRAPH_TRANSITIONS') }}
-          </dd>
-        </div>
+        <dt class="visually-hidden">
+          {{ sprintf('COM_WORKFLOW_GRAPH_STATUS', workflow.published ? 'COM_WORKFLOW_GRAPH_ENABLED' : 'COM_WORKFLOW_GRAPH_DISABLED') }}
+        </dt>
+        <dd class="me-3 mb-1 d-flex">
+          <span
+            class="badge"
+            :class="workflow.published ? 'bg-success' : 'bg-warning'"
+            role="status"
+          >
+            {{ workflow.published ? translate('COM_WORKFLOW_GRAPH_ENABLED') : translate('COM_WORKFLOW_GRAPH_DISABLED') }}
+          </span>
+        </dd>
+
+        <dt class="visually-hidden">
+          {{ sprintf('COM_WORKFLOW_GRAPH_STAGE_COUNT', stagesCount) }}
+        </dt>
+        <dd class="me-3 mb-1 d-flex">
+          {{ stagesCount }} {{ stagesCount === 1 ? translate('COM_WORKFLOW_GRAPH_STAGE') : translate('COM_WORKFLOW_GRAPH_STAGES') }}
+        </dd>
+
+        <dt class="visually-hidden">
+          {{ sprintf('COM_WORKFLOW_GRAPH_TRANSITION_COUNT', transitionsCount) }}
+        </dt>
+        <dd class="me-3 mb-1 d-flex">
+          {{ transitionsCount }} {{ transitionsCount === 1 ? translate('COM_WORKFLOW_GRAPH_TRANSITION')
+            : translate('COM_WORKFLOW_GRAPH_TRANSITIONS') }}
+        </dd>
       </dl>
     </div>
     <div