-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Revamp the Extending, Plugins, Architecture, and Programmatic configuration pages
#2696
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
Conversation
Co-authored-by: Piotr P. Karwasz <[email protected]>
Co-authored-by: Piotr P. Karwasz <[email protected]>
# Conflicts: # src/site/antora/modules/ROOT/pages/manual/customloglevels.adoc
# Conflicts: # src/site/antora/modules/ROOT/examples/manual/customloglevels/log4j2.json # src/site/antora/modules/ROOT/pages/manual/customloglevels.adoc
Editing a Ditaa source requires significant Ascii art skills, which makes it difficult to modify. This contradicts with our initial motivation to move away from images. Switching to PlantUML instead, which is a no-brainer to use and adapt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for revamping all these pages!
For now I reviewed only customconfig.adoc. Given the quality of the original, full of wrong or deprecated examples, I see a net improvement in the page.
Yet there are still some corrections to do.
src/site/antora/modules/ROOT/examples/manual/customconfig/ExampleConfigurationFactory.java
Outdated
Show resolved
Hide resolved
src/site/antora/modules/ROOT/examples/manual/customconfig/ExampleConfigurationFactory.java
Show resolved
Hide resolved
The browser does not send the fragment part to the server, when requesting a page. So all redirects from `a.html#fragment` to `b.html` will not work. On the other hand you can redirect `a.html` to `b.html#fragment`, but you need to add the NE (disables escaping) flag.
Co-authored-by: Piotr P. Karwasz <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Regarding the Extending page, I have a couple of comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Plugins page seems really nice! 💯
Co-authored-by: Piotr P. Karwasz <[email protected]>
This PR addresses #2530.