[PATCH] D65030: [clang-doc] Add second index for sections within info's content

Mon Aug 5 19:51:27 PDT 2019

DiegoAstiazaran added inline comments.

================
Comment at: clang-tools-extra/unittests/clang-doc/HTMLGeneratorTest.cpp:47-52
+<ul>
+  <li><span><a href="#Namespaces">Namespaces</a></span></li>
+  <li><span><a href="#Records">Records</a></span></li>
+  <li><span><a href="#Functions">Functions</a></span><ul>
+  <li><span><a href="#0000000000000000000000000000000000000000">OneFunction</a></span></li>
+</ul></li>
----------------
juliehockett wrote:
> DiegoAstiazaran wrote:
> > juliehockett wrote:
> > > Formatting is a bit weird for sublists
> > Fixed by D65005. It will be properly formatted after rebasing.
> This formatting is still a bit weird :( The inner `ul` didn't get its own newline, so it's hard to tell which list each item is part of. Not a huge problem, but if it's a reasonably small fix I'd love to see it cleaned up.
I had not rebased.
I rebased in the last version of this patch and the index is rendered as D65005.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65030/new/

https://reviews.llvm.org/D65030