fix: Raw style copy

surajshetty3416 · surajshetty3416 · commit 0b60cf28a38e · 2024-01-11T16:16:24.000+05:30
diff --git a/frontend/src/builder.d.ts b/frontend/src/builder.d.ts
@@ -5,7 +5,7 @@ declare interface BlockStyleMap {
 }
 
 declare interface BlockAttributeMap {
-	[key: string]: string;
+	[key: string]: string | number | null | undefined;
 }
 
 declare interface BlockOptions {
diff --git a/frontend/src/components/BlockEditor.vue b/frontend/src/components/BlockEditor.vue
@@ -114,6 +114,7 @@ watchEffect(() => {
 	props.block.getStyle("bottom");
 	props.block.getStyle("right");
 	props.block.getStyle("position");
+	props.block.rawStyles;
 	const parentBlock = props.block.getParentBlock();
 	parentBlock?.getStyle("display");
 	parentBlock?.getStyle("justifyContent");
diff --git a/frontend/src/components/ObjectEditor.vue b/frontend/src/components/ObjectEditor.vue
@@ -74,7 +74,7 @@ const pasteObj = (e: ClipboardEvent) => {
 		const map = new Map(Object.entries(props.obj));
 		try {
 			const objString = text.match(/{[^{}]+}/)?.[0];
-			if (!objString) return new Error("Invalid object");
+			if (!objString) throw new Error("Invalid object");
 			const obj = new Function("return " + objString)();
 			if (typeof obj === "object") {
 				for (const [key, value] of Object.entries(obj)) {
@@ -91,7 +91,6 @@ const pasteObj = (e: ClipboardEvent) => {
 			}
 			map.delete("");
 		}
-		e.preventDefault();
 		emit("update:obj", mapToObject(map));
 	}
 };
diff --git a/frontend/src/utils/block.ts b/frontend/src/utils/block.ts
@@ -33,7 +33,7 @@ class Block implements BlockOptions {
 	referenceBlockId?: string;
 	isRepeaterBlock?: boolean;
 	visibilityCondition?: string;
-	customAttributes?: BlockAttributeMap;
+	customAttributes: BlockAttributeMap;
 	constructor(options: BlockOptions) {
 		this.element = options.element;
 		this.innerHTML = options.innerHTML;
@@ -409,7 +409,12 @@ class Block implements BlockOptions {
 		return store.findParentBlock(this.blockId);
 	}
 	canHaveChildren(): boolean {
-		return (this.isContainer() || this.isRoot() || this.isDiv()) && !this.isExtendedFromComponent();
+		return !(
+			this.isImage() ||
+			this.isSVG() ||
+			(this.isText() && !this.isLink()) ||
+			this.isExtendedFromComponent()
+		);
 	}
 	updateStyles(styles: BlockStyleObjects) {
 		this.baseStyles = Object.assign({}, this.baseStyles, styles.baseStyles);
diff --git a/frontend/src/utils/blockController.ts b/frontend/src/utils/blockController.ts
@@ -110,15 +110,25 @@ const blockController = {
 	},
 	setRawStyles: (rawStyles: BlockStyleMap) => {
 		store.selectedBlocks.forEach((block) => {
-			block.rawStyles = rawStyles;
+			Object.keys(block.rawStyles).forEach((key) => {
+				if (!rawStyles[key]) {
+					delete block.rawStyles[key];
+				}
+			});
+			Object.assign(block.rawStyles, rawStyles);
 		});
 	},
 	getCustomAttributes: () => {
 		return blockController.isBLockSelected() && store.selectedBlocks[0].customAttributes;
 	},
 	setCustomAttributes: (customAttributes: BlockAttributeMap) => {
 		store.selectedBlocks.forEach((block) => {
-			block.customAttributes = customAttributes;
+			Object.keys(block.customAttributes).forEach((key) => {
+				if (!customAttributes[key]) {
+					delete block.customAttributes[key];
+				}
+			});
+			Object.assign(block.customAttributes, customAttributes);
 		});
 	},
 	getParentBlock: () => {

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ declare interface BlockStyleMap {`
`5`	`5`	`}`
`6`	`6`
`7`	`7`	`declare interface BlockAttributeMap {`
`8`		`- [key: string]: string;`
	`8`	`+ [key: string]: string \| number \| null \| undefined;`
`9`	`9`	`}`
`10`	`10`
`11`	`11`	`declare interface BlockOptions {`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ const pasteObj = (e: ClipboardEvent) => {`
`74`	`74`	`const map = new Map(Object.entries(props.obj));`
`75`	`75`	`try {`
`76`	`76`	`const objString = text.match(/{[^{}]+}/)?.[0];`
`77`		`- if (!objString) return new Error("Invalid object");`
	`77`	`+ if (!objString) throw new Error("Invalid object");`
`78`	`78`	`const obj = new Function("return " + objString)();`
`79`	`79`	`if (typeof obj === "object") {`
`80`	`80`	`for (const [key, value] of Object.entries(obj)) {`
`@@ -91,7 +91,6 @@ const pasteObj = (e: ClipboardEvent) => {`
`91`	`91`	`}`
`92`	`92`	`map.delete("");`
`93`	`93`	`}`
`94`		`- e.preventDefault();`
`95`	`94`	`emit("update:obj", mapToObject(map));`
`96`	`95`	`}`
`97`	`96`	`};`