Merge pull request #4 from aw2934/handleSeparators

Linhatran · web-flow · commit 3e6a98d0283b · 2021-01-21T13:16:05.000-06:00
fix issue where elements dropped onto empty divs disappear
diff --git a/app/src/helperFunctions/manageSeparators.ts b/app/src/helperFunctions/manageSeparators.ts
@@ -8,7 +8,7 @@ const manageSeparators = {};
 manageSeparators.nextTopSeparatorId = initialState.nextTopSeparatorId;
 
 manageSeparators.handleSeparators = (arr, str) => {
-  if ((str === 'delete' || str === 'change position') && arr.length === 1) {
+  if ((str === 'delete' || str === 'change position') && arr.length === 1 && arr[0].name === 'separator') {
     arr.splice(0, 1);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ const manageSeparators = {};`
`8`	`8`	`manageSeparators.nextTopSeparatorId = initialState.nextTopSeparatorId;`
`9`	`9`
`10`	`10`	`manageSeparators.handleSeparators = (arr, str) => {`
`11`		`- if ((str === 'delete' \|\| str === 'change position') && arr.length === 1) {`
	`11`	`+ if ((str === 'delete' \|\| str === 'change position') && arr.length === 1 && arr[0].name === 'separator') {`
`12`	`12`	`arr.splice(0, 1);`
`13`	`13`	`}`
`14`	`14`