On Fri, 02 Sep 2016, Eric Holscher <e...@ericholscher.com> wrote:
> This uses the proper Sphinx add_stylesheet API for adding style overrides:
>
> http://www.sphinx-doc.org/en/stable/extdev/appapi.html#sphinx.application.Sphinx.add_stylesheet
>
> Previously,
> if a user had any other `css_files` defined by extensions,
> they would be overwritten by the `html_context` redefinition here.
Removing html_context makes sense, but we don't seem to actually have
_static/theme_overrides.css, or _static directory for that matter, so
could we just drop the setup function altogether?
BR,
Jani.
>
> Signed-off-by: Eric Holscher <e...@ericholscher.com>
> ---
> Documentation/conf.py | 14 ++++++++------
> 1 file changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/Documentation/conf.py b/Documentation/conf.py
> index 96b7aa6..1707193 100644
> --- a/Documentation/conf.py
> +++ b/Documentation/conf.py
> @@ -179,12 +179,6 @@ except ImportError:
>
> html_static_path = ['sphinx-static']
>
> -html_context = {
> - 'css_files': [
> - '_static/theme_overrides.css',
> - ],
> -}
> -
> # Add any extra paths that contain custom files (such as robots.txt or
> # .htaccess) here, relative to this directory. These files are copied
> # directly to the root of the documentation.
> @@ -419,3 +413,11 @@ pdf_documents = [
> # line arguments.
> kerneldoc_bin = '../scripts/kernel-doc'
> kerneldoc_srctree = '..'
> +
> +
> +def setup(app):
> + """
> + This is a basic Sphinx extension that adds our CSS overrides.
> + """
> + app.add_stylesheet('_static/theme_overrides.css')
> +
--
Jani Nikula, Intel Open Source Technology Center
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
