-
Notifications
You must be signed in to change notification settings - Fork 327
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
New Parser/Renderer #1093
Comments
I saw mention somewhere that the new version won't allow direct html, and for awhile i've been wondering if that is 100% true, or if it just won't allow some html? Can you confirm either way, and what html tags might remain available (or become not available)? |
We will still allow HTML. I think there are way too many people out there who love the Homebrewery precisely because of how customizable it is (including me). However, we will no longer "support" raw HTML. Meaning, if something breaks because you have bad HTML, we aren't going prioritize pushing out code fixes to make sure your custom thing works (and if an update breaks your custom thing, we warned you). We have gone a long way towards preventing random crashes, etc. from custom HTML, but at some point it's beyond the scope of the Homebrewery's goals. Plus, if we can encourage and supply everything 95% of people need just with Markdown, I think that lowers the barrier to entry for a lot of people who don't necessarily know the difference between HTML and Markdown or any computer language at all. |
Is this 2017 post by Stolksdorf basically the foundation of the new renderer? The curly braces "divs" and such? Just happened across it and thought it is a well reasoned argument. Particularly the concern about moving from "photoshop expertise to coding expertise" and the issues around that. Maintaining divs was my biggest concern, and the "blocks" will do that. If this is the route forward, that is good to know because I will start to think about writing my guides (and possible snippets) with that in mind. |
Yep, that's where it comes from. He even posted 90% of the work already here on Github, but just didn't quite get the final bits finished to publish it. There are a lot of good reasons to make the changes. Now, my understanding is he wanted to fully remove any HTML capabilities, which we could do, but I don't know if such a drastic restriction is for the best. There are a few cases such as with What we might start with, though, is removing the janky code that renders markdown inside of other HTML. Normally, |
I see that "class table" is ticked as complete on the above list-- i just inserted a class table in v3 and it still comes up using divs rather than moustache syntax. just fyi. |
ugh, sorry, ignore me....i think i looked at monster stat blocks by mistake. |
Close this bad boy out? Cover Page snippet never got done, but let's lump that together with a new issue regarding "Page Layout" snippets as mentioned in #781. Leaving to @calculuschild for the cathartic release. |
...yeah... I guess... It's done isn't it? Ooo-weee. Yep, let's move cover page to the new issue and close this out. |
Future updates and fixes for current bugs is becoming more difficult due to the current structure of the Markdown->HTML->Render pipeline. This is partly due to unforseen consequences of early design choices, layers of patches and band-aids, and users with custom styling and HTML. We propose creating a new "V2" pipeline which will become the new default, leaving behind the current implementation as a "legacy" option so as to preserve older brews, but limiting further updates on that path. We have learned a lot as the Homebrewery has grown and become more popular, and it is time to put it to use in a new, better version.
This issue will collect the sub-tasks and related issues.
Note: @stolksdorf did
a lot ofalmost all of this work already, and even totally cleaned up and organized everything. The NewRenderer branch is a goldmine. Some of it might not be compatible with the current codebase, but I really like what he's done.Tasks
Create new render pipeline, "V3"
Restructure PHB Styling
Other issues fixed by this rework
The text was updated successfully, but these errors were encountered: