-
Notifications
You must be signed in to change notification settings - Fork 16
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
Are there plans to support phpBB themes? #48
Comments
this is not so easy, because there is no standardized handling for the phpbb templates also they all need to be customized to work with lemmy |
The problem is that each phpBB theme includes its own HTML template files, which can be completely distinct. If you want to make a theme work with lemmybb, you need to follow these steps:
The last step is obviously the most complicated one. Filenames in templates folder are hardcoded in lemmybb, so you cant change them, and they dont relate to phpbb file structure because I took the HTML from a running phpbb site, not from the raw templates files. Basically there will be some trial and error to get things working. If you want to give it a try, I suggest you make a pull request with your changes, so others can follow along and help out. Once its working and if the theme is open source, we can include it with lemmybb to give some different choices to users. |
How stable do you feel the codebase is right now for things like element names? I am very excited about this project but I want to wait until things are fleshed out so I don't wind up doing a lot of work twice. |
Most of what you see in the templates are lines like Btw if we can merge your theme into this repo, it should be easy to add test cases which detect breakage. That would help to stabilize the template rendering and make it easier to develop more themes. |
Question in title, apologizes if this has been answered somewhere.
Edit: I see this has been mentioned here: #24
We should probably add some official mention around this though as it will be a highly requested feature.
The text was updated successfully, but these errors were encountered: