[PATCH] D12129: [libcxx] Add new Sphinx documentation

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 18 18:39:03 PDT 2015


EricWF added a comment.

In http://reviews.llvm.org/D12129#227320, @silvas wrote:

> From a Sphinx perspective, this looks fine to me. You may want to consider using `html_theme = 'haiku'` for consistency with clang, but reusing LLVM's is fine. (the reason I had clang use 'haiku' is that it is a bit better put together since it ships with sphinx (llvm's theme also has a notorious problem distinguishing different header levels); 'haiku' also doesn't require any static theme-related assets to be checked in since sphinx has them already as part of its own installation)


I'll take a look at using clang's theme and see what comes out. Ironically I think I chose LLVM's theme because I preferred the way it distinguished between headers levels (although I have noticed some issues distinguishing nested levels). It would be nice to remove some assets from the repository.


http://reviews.llvm.org/D12129





More information about the cfe-commits mailing list