Open duncanmmacleod opened 2 years ago
The html_minify option uses a custom site_pages list to know what documents to minify, which is built as follows:
html_minify
site_pages
https://github.com/bashtage/sphinx-material/blob/3c3d1533a8ef09dad6987e06fc5375867c18c529/sphinx_material/__init__.py#L60-L61
which doesn't work when using the dirhtml builder, for which the page URL is something like
dirhtml
os.path.join(app.outdir, pagename + "/index.html")
...I think. If it's as easy as checking the builder in an if statement, I might be able to put together a patch.
if
A PR is welcome. Thanks.
bashtage
commented
2 years ago bashtage
commented
2 years ago
The
html_minifyoption uses a customsite_pageslist to know what documents to minify, which is built as follows:https://github.com/bashtage/sphinx-material/blob/3c3d1533a8ef09dad6987e06fc5375867c18c529/sphinx_material/__init__.py#L60-L61
which doesn't work when using the
dirhtmlbuilder, for which the page URL is something like...I think. If it's as easy as checking the builder in an
ifstatement, I might be able to put together a patch.