Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Fix duplicated application layout #127

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Fix duplicated application layout #127

merged 1 commit into from
Mar 1, 2024

Conversation

cjdenio
Copy link
Contributor

@cjdenio cjdenio commented Mar 1, 2024

The Problem

View the source for the homepage. There's an <html> inside an <html>!!!

Screenshot 2024-02-29 at 7 34 52 PM

rails_nestable_layouts automatically uses the application layout as the root, so specifying it in nested_layouts causes it to be rendered twice.

@reesericci
Copy link
Member

How have I never noticed this, thanks for the PR!

@reesericci reesericci merged commit 851ae89 into obl-ong:main Mar 1, 2024
1 check failed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants