Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11209 )

Change subject: [site] add links required by ASF
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11209/2/_includes/top_common.html
File _includes/top_common.html:

http://gerrit.cloudera.org:8080/#/c/11209/2/_includes/top_common.html@93
PS2, Line 93:                 <li><a href="https://www.apache.org/security/"; 
target="_blank">Security</a></li>
> It feels a bit weird to me to create a landing page to list 4 links so mobi
yeah, I understand that. Thats an unfortunate effect of our mobile drop down 
choices I suppose.

I think 4 top level links for things that are not very commonly used is also 
not great. Maybe we could add an Apache section to the existing Community drop 
down and add the link to the community page too?



--
To view, visit http://gerrit.cloudera.org:8080/11209
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: gh-pages
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6983314e598eff73ac8f476924acd9d6f62196f
Gerrit-Change-Number: 11209
Gerrit-PatchSet: 2
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Comment-Date: Wed, 15 Aug 2018 14:34:24 +0000
Gerrit-HasComments: Yes

Reply via email to