Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Separate "embed theme" settings #2262

Closed
wants to merge 5 commits into from
Closed

Separate "embed theme" settings #2262

wants to merge 5 commits into from

Conversation

daedeloth
Copy link

These changes add a new tab to the settings page "embed theme" (only visible when embedding is enabled).
Works very similar to the mobile theme.

A query parameter embed=1 is added to determine if the forum is embeded.
(As far as I understand from online documentation, this should not be a problem seo wise)

Working implementation:
http://catlab.be/quized-development-forum-holiday-release (embedded)
http://forum.catlab.eu/ (bootstrap theme)

More info:
http://vanillaforums.org/discussion/28467/separate-embed-theme-setting#latest

@linc
Copy link
Contributor

linc commented Jan 21, 2015

Howdy @daedeloth, I need you to sign our CLA before we can look at this. http://vanillaforums.org/contributors You'll need an account on vanilaforums.org to do that.

This would be better targeted at the stage branch since it's a feature addition that would need additional testing before going to master. Including more details about what issue(s) you're attempting to solve with this PR would be helpful as well.

Given all that overhead, I am going to close this for now, but please feel free to resubmit per my notes here.

@daedeloth
Copy link
Author

I have made a new pull request: #2579

The issue I am trying to solve is that I want to have a forum that is available online (and looks good) + a forum that is embedded in my website (and looks good overthere as well). For me, this problem could be solved by having a seperate "embed" theme, that is displayed on the website, while the default theme is still used when accessing the forum directly.

These changes solve my problem, but I also believe this is something that could be used by anyone who's embedding their forum into their website.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants