----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5765/#review9012 -----------------------------------------------------------
Thanks for the site Ralph. Overall the site looks great for a first cut. Some feedback for your consideration: * The license link on the side displays the license header. It should instead display or link to the normative AL v2.0 * Just like the Releases on the left side, the Documentation should also expand to itemize the releases and provide links to the static release documents. * The "How to get involved" details currently show up underneath the release 1.2.0 menu item on the left. Considering this generally applies to the entire project, it merits its own menu heading with necessary details inside. * The developer access under source repository mentions log4j instead of flume ($ svn checkout https://svn.apache.org/repos/asf/flume/trunk log4j) * Although the colors are bright and of high contrast, I feel it may be better to tone them down and use more of white/gray or other milder shades if possible. Regards, Arvind Prabhakar - Arvind Prabhakar On July 8, 2012, 7:36 p.m., Ralph Goers wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5765/ > ----------------------------------------------------------- > > (Updated July 8, 2012, 7:36 p.m.) > > > Review request for Flume. > > > Description > ------- > > This contains the source to build the initial version of the web site using > the CMS, Maven and Sphinx. To test the build just run mvn site and the > output of the site will be in target/site. "mvn -P pdf package" is supposed > to package the users and developers guides as pdf's to be deployed to the > site as part of a release but that isn't quite working yet. > > A few notes: > 1. The site will be committed to > https://svn.apache.org/repos/asf/flume/site/trunk. > 2. The site is incomplete in that it is missing release information. This > will be directly added once the site is published to the production svn > location. > > > This addresses bug FLUME-813. > https://issues.apache.org/jira/browse/FLUME-813 > > > Diffs > ----- > > > Diff: https://reviews.apache.org/r/5765/diff/ > > > Testing > ------- > > > Thanks, > > Ralph Goers > >
