payfrit/payfrit-works
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix modifier move: use Shift key and dataTransfer data instead of effectAllowed

Browse source
This commit is contained in:
John Mizerek 2026-01-14 12:57:34 -08:00
parent ff6e4f2a36
commit f40d5c1f93
1 changed files with 7 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
portal/menu-builder.html
View file

@ -4,6 +4,7 @@
<meta charset="UTF-8"> <meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0"> <meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Menu Builder - Payfrit</title> <title>Menu Builder - Payfrit</title>
<link rel="icon" type="image/svg+xml" href="favicon.svg">
<link rel="stylesheet" href="portal.css"> <link rel="stylesheet" href="portal.css">
<style> <style>
/* Menu Builder Specific Styles */ /* Menu Builder Specific Styles */
@ -3263,8 +3264,10 @@
e.dataTransfer.setData('parentId', card.dataset.parentId); e.dataTransfer.setData('parentId', card.dataset.parentId);
e.dataTransfer.setData('depth', card.dataset.depth); e.dataTransfer.setData('depth', card.dataset.depth);
e.dataTransfer.setData('source', 'modifier'); e.dataTransfer.setData('source', 'modifier');
// Default is copy, hold Ctrl+Alt for move // Default is copy, hold Shift for move
e.dataTransfer.effectAllowed = (e.ctrlKey && e.altKey) ? 'move' : 'copy'; const isMove = e.shiftKey;
e.dataTransfer.setData('isMove', isMove ? 'true' : 'false');
e.dataTransfer.effectAllowed = isMove ? 'move' : 'copy';
card.classList.add('dragging'); card.classList.add('dragging');
}); });
@ -3302,7 +3305,7 @@
const draggedParentId = e.dataTransfer.getData('parentId'); const draggedParentId = e.dataTransfer.getData('parentId');
const targetModId = card.dataset.modifierId; const targetModId = card.dataset.modifierId;
const targetParentId = card.dataset.parentId; const targetParentId = card.dataset.parentId;
const isMove = e.dataTransfer.effectAllowed === 'move'; const isMove = e.dataTransfer.getData('isMove') === 'true';
if (draggedModId && targetModId && draggedModId !== targetModId) { if (draggedModId && targetModId && draggedModId !== targetModId) {
const position = card.classList.contains('drop-before') ? 'before' : 'after'; const position = card.classList.contains('drop-before') ? 'before' : 'after';
@ -3347,7 +3350,7 @@
const draggedModId = e.dataTransfer.getData('modifierId'); const draggedModId = e.dataTransfer.getData('modifierId');
const draggedParentId = e.dataTransfer.getData('parentId'); const draggedParentId = e.dataTransfer.getData('parentId');
const targetItemId = card.dataset.itemId; const targetItemId = card.dataset.itemId;
const isMove = e.dataTransfer.effectAllowed === 'move'; const isMove = e.dataTransfer.getData('isMove') === 'true';
if (draggedModId && targetItemId) { if (draggedModId && targetItemId) {
if (isMove) { if (isMove) {