aem-core-wcm-components icon indicating copy to clipboard operation
aem-core-wcm-components copied to clipboard

Core Container - Responsive Grid including through the Sightly data-sly-resource

Open Daniil-Zaitsau opened this issue 4 years ago • 3 comments

Bug Report

Current Behavior Core Container Component, which was added with the data-sly-resource, does not have the ability to add components inside if the responsive layout is selected in policies.

Expected behavior/code I should be able to add components inside.

Environment

  • AEM 6.5 SP 6.5.6
  • Core Components version 2.13.2

Additional context / Screenshots Steps to reproduce:

  1. Create a component, that will only extend the Core Container Component (by the sling:resourceSuperType) and will not override any file image
  2. Create a component, that will include a component from the first stage on the test page using data-sly-resource image
  3. Add a component from the second step to the test page image
  4. Go to the page template editor image
  5. Open the design dialog for the component from the first step (container) image
  6. Go to the 'Container Settings' tab and select the Responsive Grid as the layout value image
  7. Return to the test page. The component has lost the ability to add other components inside image

Daniil-Zaitsau avatar Jan 04 '21 20:01 Daniil-Zaitsau

@adobe export issue to Jira Project SITES as Bug

bpauli avatar Mar 15 '22 08:03 bpauli

:white_check_mark: Jira issue SITES-5384 is successfully created for this GitHub issue.

github-jira-sync-bot avatar Mar 15 '22 08:03 github-jira-sync-bot

Is there a reason why the container cannot be included by path as the page component does [1], as opposed to include the container by resource? Without a use-case, there's no clear reason why this should be necessary.

[1] https://github.com/adobe/aem-core-wcm-components/blob/main/content/src/content/jcr_root/apps/core/wcm/components/page/v3/page/body.html#L18

gabrielwalt avatar Mar 23 '22 09:03 gabrielwalt

Closing this issue as mentioned by @gabrielwalt the container can be included by path

bpauli avatar Oct 19 '22 07:10 bpauli