Refactor for better performance and property naming

Author: deedeeh

dotcom-rendering/src/layouts/GalleryLayout.tsx

@@ -292,7 +292,7 @@ export const GalleryLayout = (props: WebProps | AppProps) => {
 						) : null}
 					</div>
 				</header>
-				{gallery.images.map((element, index) => {
+				{gallery.bodyElements.map((element, index) => {
 					const isImage =
 						element._type ===
 						'model.dotcomrendering.pageElements.ImageBlockElement';

dotcom-rendering/src/model/enhance-ad-placeholders.ts

@@ -106,15 +106,13 @@ const insertAdPlaceholdersForGallery = (elements: FEElement[]): FEElement[] => {
 
 			const shouldInsertAd = imageBlockElementCounter % 4 === 0;
 
-			const currentElements = [...prev.elements, currentElement];
-
 			return {
 				elements: shouldInsertAd
 					? insertPlaceholderAfterCurrentElement(
 							prev.elements,
 							currentElement,
 					  )
-					: currentElements,
+					: [...prev.elements, currentElement],
 				imageBlockElementCounter,
 			};
 		},

dotcom-rendering/src/types/article.ts

@@ -45,7 +45,7 @@ export type ArticleFields = {
 
 export type Gallery = ArticleFields & {
 	design: ArticleDesign.Gallery;
-	images: (ImageBlockElement | AdPlaceholderBlockElement)[];
+	bodyElements: (ImageBlockElement | AdPlaceholderBlockElement)[];
 	mainMedia: ImageBlockElement;
 };
 
@@ -135,7 +135,7 @@ export const enhanceArticleType = (
 			design,
 			display: format.display,
 			theme: format.theme,
-			images: blocks.flatMap((block) =>
+			bodyElements: blocks.flatMap((block) =>
 				block.elements.filter(
 					(element) =>
 						element._type ===