refactor: remove redundant logic for isDraggableHandle

primefaces · Aug 15, 2024 · 4cc6866 · 4cc6866
1 parent e67641d
commit 4cc6866
Showing 1 changed file with 5 additions and 18 deletions.
diff --git a/components/lib/datatable/TableBody.js b/components/lib/datatable/TableBody.js
@@ -592,25 +592,12 @@ export const TableBody = React.memo(
  const onRowMouseDown = (e) => {
  const { originalEvent: event } = e;
 
- if (isUnstyled()) {
- const isDraggableHandle = DomHandler.getAttribute(event.target, 'data-pc-section') === 'rowreordericon' || event.target.closest('[data-pc-section="rowreordericon"]');
+ const isDraggableHandle = isUnstyled()
+ ? DomHandler.getAttribute(event.target, 'data-pc-section') === 'rowreordericon' || event.target.closest('[data-pc-section="rowreordericon"]')
+ : DomHandler.hasClass(event.target, 'p-datatable-reorderablerow-handle') || event.target.closest('.p-datatable-reorderablerow-handle');
 
- if (isDraggableHandle) {
- event.currentTarget.draggable = true;
- event.target.draggable = false;
- } else {
- event.currentTarget.draggable = false;
- }
- } else {
- const isDraggableHandle = DomHandler.hasClass(event.target, 'p-datatable-reorderablerow-handle') || event.target.closest('.p-datatable-reorderablerow-handle');
-
- if (isDraggableHandle) {
- event.currentTarget.draggable = true;
- event.target.draggable = false;
- } else {
- event.currentTarget.draggable = false;
- }
- }
+ event.currentTarget.draggable = isDraggableHandle;
+ event.target.draggable = !isDraggableHandle;
 
  if (allowRowDrag(e)) {
  enableDragSelection(event, 'row');