Skip to content

Footer hidden on specific pages#1

Merged
Devonab merged 3 commits intoDevonab:mainfrom
holmesadam:hidden-on-pages
Jan 7, 2025
Merged

Footer hidden on specific pages#1
Devonab merged 3 commits intoDevonab:mainfrom
holmesadam:hidden-on-pages

Conversation

@holmesadam
Copy link
Contributor

The plugin has functionality to "hide on auth pages", however, the paths set in the auth constant are not always the auth pages.
This PR allows the user to set their own array of pages to ignore the footer on (not just auth pages). See readme for methods.

@Devonab Devonab added the enhancement New feature or request label Jan 6, 2025
@Devonab Devonab self-assigned this Jan 6, 2025
- Test default footer visibility
- Test footer visibility on non-configured pages
- Test footer hiding on configured pages
@Devonab Devonab merged commit b001fc4 into Devonab:main Jan 7, 2025
9 checks passed
@Devonab
Copy link
Owner

Devonab commented Jan 7, 2025

Thank you for your contribution 😊
It's a good idea to be able to specify the pages for which we want to hide the footer.

@holmesadam
Copy link
Contributor Author

Thank you! And apologies for not adding tests 🤦🏻‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants