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
[JAMES-3226] drops unused ui-bundle #13
Conversation
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.
Shouldn't we drop /antora-ui
too?
I think we should keep antora-ui for now I am experimenting with different schemes for modifying the UI I'll do a followup cleanup PR once I have something working in a satisfying way |
Antora supplemental_files are not well documented but they allow to override the UI templates from the UI bundle, not that this mecanisme DOES NOT allow properly overriding UI style, only content and HTML structure.
@chibenwa I updated this PR with a couple commits demonstrating how to customize UI, the demo is a proof of concept I didn't spend too much time on styling the icons for example, some of the links are not perfect but it shows how it's done :) |
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.
Can you share a screenshot?
ideally I would prefer to have this merged before merging to live and hard resetting staging :) |
Yes, I did not want to hurry too ;uch to give a chance for over people to review this. I am unsure how many people are actively watching this repository. I will merge it now, it is several days already. |
It seems that an effort to use an customized UI bundle was started and abandonned halfway. The bundle currently isn't used in the website generation and can confuse contributors.
I propose to remove them, since the modifications it contain can now be replicated by using antora supplementary files to override the default UI