jdaugherty commented on code in PR #15710:
URL: https://github.com/apache/grails-core/pull/15710#discussion_r3341345552
##########
grails-forge/grails-forge-core/src/main/java/org/grails/forge/feature/sitemesh3/Sitemesh3.java:
##########
@@ -22,14 +22,20 @@
import org.grails.forge.application.ApplicationType;
import org.grails.forge.application.generator.GeneratorContext;
import org.grails.forge.build.dependencies.Dependency;
-import org.grails.forge.feature.Category;
+import org.grails.forge.feature.DefaultFeature;
import org.grails.forge.feature.Feature;
+import org.grails.forge.feature.view.GspLayout;
+import org.grails.forge.options.Options;
-@Singleton
-public class Sitemesh3 implements Feature {
+import java.util.Set;
- public Sitemesh3() {
- }
+/**
+ * Default GSP layout decorator, backed by SiteMesh 3 ({@code
grails-sitemesh3}).
+ * Applied automatically to web applications unless another {@link GspLayout}
+ * (e.g. {@code grails-layout}) is explicitly selected.
+ */
+@Singleton
+public class Sitemesh3 extends GspLayout implements DefaultFeature {
Review Comment:
I think making this the default & updating to sitemesh 3 should be separate
PRs
--
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]
