Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[templates/nextjs-sxa]: The background image in Container component has been fixed #SXA-7647 #1997

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

sc-ruslanmatkovskyi
Copy link
Collaborator

@sc-ruslanmatkovskyi sc-ruslanmatkovskyi commented Dec 9, 2024

Description / Motivation

Fixed the background-image issue in the Container component for the pageState === 'edit' scenario.

Implemented correct parsing and formatting of the BackgroundImage parameter to ensure proper rendering during the edit mode.

Replaced UUID dashes with an empty string and prefixed the image path with /sitecore/shell/-/media/.

Extracted the background image processing logic into a reusable function getBackgroundStyle for better code organization and clarity.

Testing Details

  • Unit Test Added
  • Manual Test/Other (Please elaborate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the bugfix/SXA-7647-background-image-container-component branch from 474547b to 2fed8c9 Compare December 11, 2024 13:46
@sc-ruslanmatkovskyi sc-ruslanmatkovskyi force-pushed the bugfix/SXA-7647-background-image-container-component branch from 2fed8c9 to 2eaed8b Compare December 11, 2024 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants