-
Notifications
You must be signed in to change notification settings - Fork 20
A redesign of my theme #26
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
|
This may be shipped with the next version of apigen. |
|
@olvlvl I'm concerned of how the user is able to open up the source. I tried in your demo but I wasn't able to open up the source of a specific method. |
|
I have to look into this because I'm generating my docs without sources. |
|
Hi @olvlvl , thank you for great work! We are merging themes to main ApiGen\ApiGen repository, already did for DefaultTheme, so it's easier to develop. Could you send PR there after you finish it here? I'll take care about the changes in v5, don't worry about them. |
|
Can I add this? |
|
Hi Tomas, give me a sec, I'll do it. |
|
Here: ApiGen/ApiGen#840 |
|
Awesome job. Thank you! |
Hi,
I redesigned my bootstrap theme, improving search suggestions, sidebar overflow, legibility, and I also replaced the bootstrap v3 styles with a selection of v4 (I removed the big bootstrap file).
Biggest changes:
You can see it in action over here: https://olvlvl.com/apigen/class-ApiGen.Parser.Reflection.ReflectionElement.html
Feedback appreciated <3