Skip to content

Conversation

@smg6511
Copy link
Collaborator

@smg6511 smg6511 commented Nov 30, 2025

What does it do?

Adds missing regions to FC set schema and renames those regions’ IDs to be more consistent with current naming convention.

Why is it needed?

It's unclear that two additional regions exist above and below the main content field. Also, even though you can currently assign TVs to these regions, you can not control the overall visibility of those regions.

How to test

  1. Because there's a change to a schema, a full rebuild (_build/transport.core.php) and running of the setup (install) must first be done
  2. Create at least one TV for testing
  3. Create a Form Customization set, and note that the modx-resource-content-above and modx-resource-content-below now show in the Regions tab
  4. Move a TV to both of these regions and verify they appear in the expected position in a Resource
  5. Uncheck each of these regions in the FC Set Region tab and verify each region is hidden when viewing a Resource
  6. Also, verify that usage of the legacy Region IDs work as well (modx-content-above and modx-content-below)

Related issue(s)/PR(s)

Resolves #16442

@smg6511 smg6511 added area-fc Work related to form customization rules pr/review-needed Pull request requires review and testing. labels Nov 30, 2025
,hideMode: 'offsets'
,items: [{
id: 'modx-content-above'
id: 'modx-resource-content-above'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a breaking change, I'm not sure if big enough, but can break some extras that are injecting to those places and using the ID for identifying the space. (same goes for the modx-content-below)

Copy link
Collaborator Author

@smg6511 smg6511 Dec 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It shouldn't be: There's BC support added in modActionDom that transforms the old id to the new one as the rules are actually being built. Is there a scenario I'm not thinking of that would be problematic?

Although I suppose I'm thinking of this specifically in the context of Form Customization; guessing you're thinking one might have reason to directly access these two components outside of the FC process?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@theboxer - Ok, to cover your concern I went ahead and added back in two items using the original ids with a deprecation notice.

Add back in legacy above/below ids to provide wider BC support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-fc Work related to form customization rules pr/review-needed Pull request requires review and testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Please put "modx-content-above" and "modx-content-below" in the list of regions in manager form customization

2 participants