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

Reply via email to