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

Add WidgetShell Component #4290

Closed
wants to merge 2 commits into from
Closed

Add WidgetShell Component #4290

wants to merge 2 commits into from

Conversation

HarshMN2345
Copy link

@HarshMN2345 HarshMN2345 commented Oct 1, 2023

Description
This pull request introduces a new component, WidgetShell, which acts as a wrapper for child components based on whether the application is running in an iframe or not.

Changes Made
Added a new component called WidgetShell to the codebase. Integrated WidgetShell into the existing code structure to conditionally render either TransparentShell or MockPreviewShell. Implemented conditional rendering logic based on whether the component is in an iframe. Added styles for TransparentShell, MockPreviewShell, and ShellWrapper. Added Features/Enhancements
Introduced WidgetShell to manage the rendering behavior of child components. Applied conditional rendering to adapt to different environments (iframe vs. non-iframe).
Fixed Issues-None
Testing:
The changes were thoroughly tested in the following ways:
Reviewed for potential layout issues in different scenarios (iframe vs. non-iframe). Verified that the component's behavior aligns with expected outcomes.
Known Issues/Considerations-None

Description
This pull request introduces a new component, WidgetShell, which acts as a wrapper for child components based on whether the application is running in an iframe or not.

Changes Made
Added a new component called WidgetShell to the codebase.
Integrated WidgetShell into the existing code structure to conditionally render either TransparentShell or MockPreviewShell.
Implemented conditional rendering logic based on whether the component is in an iframe.
Added styles for TransparentShell, MockPreviewShell, and ShellWrapper.
Added Features/Enhancements
Introduced WidgetShell to manage the rendering behavior of child components.
Applied conditional rendering to adapt to different environments (iframe vs. non-iframe).
Fixed Issues
None
Testing
The changes were thoroughly tested in the following ways:

Reviewed for potential layout issues in different scenarios (iframe vs. non-iframe).
Verified that the component's behavior aligns with expected outcomes.
Known Issues/Considerations
None
@scopsy
Copy link
Contributor

scopsy commented Oct 2, 2023

@HarshMN2345 please submit a reproducible issue describing what is that you are trying to solve before submitting a PR.

@scopsy scopsy closed this Oct 2, 2023
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.

2 participants