codeconsole opened a new pull request, #15713:
URL: https://github.com/apache/grails-core/pull/15713
Makes SiteMesh 3 the default GSP layout in generated applications, mutually
exclusive with the legacy SiteMesh 2 `grails-layout` feature.
Introduces a `GspLayout` one-of feature group (enforced by
`OneOfFeatureValidator`):
- **`GspLayout`** — abstract one-of parent (`Category.VIEW`, WEB/WEB_PLUGIN)
- **`Sitemesh3`** — default member; auto-applied unless another `GspLayout`
is selected; adds `grails-sitemesh3`
- **`GrailsLayout`** — opt-in member; adds `grails-layout` (SiteMesh 2)
`GrailsGsp` is intentionally **not** modified: its existing `if
(!isFeaturePresent(Sitemesh3))` guard already skips `grails-layout` when
SiteMesh 3 applies, so SiteMesh 2 remains available via `GrailsLayout`.
Selecting both `sitemesh3` and `grails-layout` now fails fast.
### Depends on #15710
Split out from #15710 per review feedback ("making this the default &
updating to sitemesh 3 should be separate PRs"). This branch is stacked on
#15710, so **until #15710 merges this PR's diff also shows the
enable-SiteMesh-3 commits**; it will reduce to just the make-default change
once #15710 lands. Please merge #15710 first.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
