Use custom-block for docstrings

[tpoisot]

The current styling for docstrings isn't really in line with the rest of the vitepress theme - this is a work in progress PR to use class='jldocstring custom-block' on the div, to make it look more native.

[tpoisot]

What is the secret sauce to get the site to build using github actions? I'm getting a final_site folder on gh-pages:

[asinghvi17]

Thanks for the PR! This looks promising.

https://tpoisot.github.io/DocumenterVitepress.jl/dev/api has some examples of what this looks like now.

From a brief look it seems that font sizes are smaller in the docstring block than in regular text?

[avik-pal]

Would it be possible to mimic the Documenter.jl style for the docstrings here? We can leave out the collapsible and such.

The border was added because DocumenterMarkdown used to generate continuous blocks of docstrings for mkdocs (making it hard to distinguish 2 separate docstrings, except the # marker) which was a constant source of complaint from users.

[lazarusA]

collapsible yes, NOT this. I was playing around with them and it just adds overhead most of times. And, in general, I do like a smaller font, probably not too small.

[tpoisot]

Thanks all -- I think collapsible (open by default) is definitely achievable, I'm working on it

Font size: this is how custom-block are styles by vitepress, but if there's a .jldocstring class this can be tweaked

[lazarusA]

You could add new styles as in here https://github.com/JuliaDataCubes/YAXArrays.jl/blob/master/docs%2Fsrc%2F.vitepress%2Ftheme%2Fstyle.css#L259

[tpoisot]

yup - that's what I'm trying to do but without success so far...

[tpoisot]

I get it... for some reason it's pulling the style.css from this repo and not from my fork - scratch this, more accurately, the style.css is compiled as style_css.md?

[asinghvi17]

Maybe it's pulling it from docs/src/.vitepress since there's already a style.css there...

[lazarusA]

well... it looks great as it is. Except, we have a huge empty space when collapsed.

[tpoisot]

I think that's done?

[avik-pal]

Any particular reason to revert the badge (9b637a4)? Personally it looked nicer compared to the dash

[lazarusA]

Oh, yeah, I merged thinking the bubble badges were there. We should bring them back.

[tpoisot]

They messed with the spacing a little, but I can try to adjust the margins and bring them back!