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

Julie Hockett via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 5 16:59:48 PDT 2019


juliehockett 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>
----------------
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.


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

https://reviews.llvm.org/D65030





More information about the cfe-commits mailing list