adobe / aem-core-wcm-components

Standardized components to build websites with AEM.
https://docs.adobe.com/content/help/en/experience-manager-core-components/using/introduction.html
Apache License 2.0
730 stars 736 forks source link

[Form Container (v2)] Policy Responsive Settings Columns property does not apply #745

Closed slocum-adobe closed 4 years ago

slocum-adobe commented 4 years ago

Bug Report

Current Behavior As an author, I navigate to the core component template and configures the policy of the Form Container (v2). I navigate to the Responsive Settings tab, and configure the number of columns to 2. I save the change. I then drag and drop the Form Container (v2) onto the core component page. The responsive columns are not applied to my form container. There is a single parsys.

Expected behavior/code As an author, I expect there to be two columns within my form container to add form components to.

Environment

JasonUSC commented 4 years ago

This bug is a blocker in our implementation so hopefully it gets fixed asap.

Update: My mistake, I did not realize this was the "form" container. We've had some problems with columns on the regular container and it may be due to changes in how the AEM grid css is being built now per the documentation in bpauli's comment below.

bpauli commented 4 years ago

@JasonUSC @slocum-adobe Actually this is not a bug :). The Form Container reuses the same functionality as the layout container. The responsive settings in the content policy does only control how many columns an author can use when they resize the components in the layout mode. For more information please have a look to the corresponding documentation for the layout container at [1]. Please make sure that you import the Responsive CSS to you own client lib as described on the same helpx page, otherwise the Layout Mode will not work.

[1] https://helpx.adobe.com/experience-manager/6-5/sites/administering/using/configuring-responsive-layout.html#ConfiguretheGridoftheLayoutContainer

gabrielwalt commented 4 years ago

This seems to work as designed: configuring a form container to 2 columns doesn't mean that 2 columns will be created when inserting a form container to the page, but it means that any container inside of it can be resized to only 2 columns (instead of 12 by default).