Add a `put_layout {MyAppWeb.LayoutView, "app.html"}` on `MyAppWeb` by default

[kelvin....@gmail.com]

So, this came from a discussion in here: https://github.com/phoenixframework/phoenix/issues/3865

 

I think even though it would make no actual difference, as if we don't do that, `put_default_views` will do it for us, explicitness would benefit everyone.

I would go even further and actually change the default value of `put_default_views` to `false` in here: https://github.com/phoenixframework/phoenix/blob/f7784f86ed803a39eb45486f0e031e974e708818/lib/phoenix/controller.ex#L173

And also deprecate the use of that option in favor of being explicit on every controller of which view to use. I know it is a breaking change, but it is one that would enforce explicitness and would make the web code less "magic".

[kelvin....@gmail.com]

Going a little bit nuts now: I would even deprecate `put_view`, `put_layout` and `render` macros in favor of being explicit of which layout and which view to use for every render, but maybe that would be too much of a change 😂 

[Chris McCord]

Every choice is a tradeoff, so it’s up to you to figure out which things you’d like to optimize for.  I think we strike a nice balance today.

--
You received this message because you are subscribed to the Google Groups "phoenix-core" group.
To unsubscribe from this group and stop receiving emails from it, send an email to phoenix-core...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/phoenix-core/4b05c08c-9888-4cf0-8c7a-66e5b1c8380b%40googlegroups.com.

[Chris McCord]

The ship has sailed on this possibility, but you are welcome to do it in your applications :)

--
You received this message because you are subscribed to the Google Groups "phoenix-core" group.
To unsubscribe from this group and stop receiving emails from it, send an email to phoenix-core...@googlegroups.com.

To view this discussion on the web visit https://groups.google.com/d/msgid/phoenix-core/5e3d4cc8-2414-4247-bbec-61554253b5f6%40googlegroups.com.

[Allen Madsen]

What about adding put_layout and put_view as explicit calls in the generators?

Allen Madsen
http://www.allenmadsen.com

To view this discussion on the web visit https://groups.google.com/d/msgid/phoenix-core/4A79D0EC-90F1-4174-B08B-F2E7E4CFEA7A%40chrismccord.com.

[Allen Madsen]

By generators I meant the controller generator, not the MyAppWeb one.

Allen Madsen

http://www.allenmadsen.com

[José Valim]

In my opinion it is unnecessary noise once you get familiar with the convention.

The only reason why put_root_layout is not also automatically done is backwards compatibility.

To view this discussion on the web visit https://groups.google.com/d/msgid/phoenix-core/CAK-y3CtkgKdHbxSLGC7udb9%2B4RK%2BkJrGvQWFJu2L_AphWvf3hg%40mail.gmail.com.