> On Oct. 18, 2017, 6:06 p.m., Andrew Schwartzmeyer wrote: > > I tested this out locally on my Mac, rebased and with my getting started > > doc changes just to test it out. It's all working as expected. Though > > perhaps instead of black, might a medium gray look better?
I said `black` so that it would be the same as the title text color. I'm happy for it to be any specific color that works. - James ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62980/#review188538 ----------------------------------------------------------- On Oct. 13, 2017, 4:58 p.m., James Peach wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62980/ > ----------------------------------------------------------- > > (Updated Oct. 13, 2017, 4:58 p.m.) > > > Review request for mesos, Greg Mann and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > This follows the example of the Middleman documentation and > uses AnchorJS to inject link anchors to all headings at page > load time. The styling is basic black to match the styling of > the header titles. > > > Diffs > ----- > > site/source/assets/css/main.css 83596ddbd833e36b60bdbbd487ebd464b3874119 > site/source/layouts/basic.erb 3bf1f55d7d8feafd3caaa7902755404f7e45bae6 > > > Diff: https://reviews.apache.org/r/62980/diff/2/ > > > Testing > ------- > > Verified manually using the Docker image to run Middleman in dev mode. > > > Thanks, > > James Peach > >