Route layout#1506
Conversation
|
I was able to make some additions to the fixture. So if you run deno task fixture, you should be able to navigate to the following paths to see it in action: Those were some of the use cases i was checking out. You can see that it has no impact on existing pages: |
|
My bad on that, forgot to rerun deno fmt after adding the fixture tests. Hopefully that resolves what i saw in the workflow error. |
|
I haven't looked at this in detail yet, but just noting that we have a similar PR: #870. If we go with this one, then we should close 870. |
marvinhagemeister
left a comment
marvinhagemeister
left a comment
There was a problem hiding this comment.
Took the liberty of adding a couple of tests. Thank you so much for adding this to Fresh ❤️
|
How should i go about toggling routeConfig wrapper layouts based on a header? 'hx-request' in my case. |
|
@bendik Can you please open a new issue instead of commenting on merged PRs? |
6 participants
This is an initial implementation of a _layout file implementation that allows for applying layouts to routes. It supports cascading _layout files in route sub directories and treats them all as optional.
There are a good number of tests that were broken when I forked, so could use some pointers on where to add additional tests for this.