[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Thanks all. ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user mmiklavc commented on the issue: https://github.com/apache/metron/pull/938 +1 from me as well, thanks @anandsubbu! ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user cestella commented on the issue: https://github.com/apache/metron/pull/938 +1 as well, looks great ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/938 +1, pending Travis. I think any look complaints after this are follow-on, given that this addresses the main issues we wanted and has the updated mark. ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Roger that. Here's the new look. Let me know if this looks ok @justinleet and @cestella .  ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/938 Yeah, I thought that was more third party thing. Admittedly, this seems at (high) risk of becoming a bikeshed. I'd probably rather just use the main mark (https://www.apache.org/foundation/press/kit/asf_logo.png), even if it's suboptimal looking but noncontroversial in terms of branding, and just fix it later if we want. ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user cestella commented on the issue: https://github.com/apache/metron/pull/938 Where is that powered by apache logo from? Are we sure it doesn't mean that the apache web server serves it up? ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Thanks @justinleet for the pointer. I have made a change to use the Powered By logo instead of the foundation logo with feather, since I thought this looked better. I also added a direct link to the ASF press kit page, instead of storing the image on our local site. Let me know your thoughts. https://user-images.githubusercontent.com/20395490/36713710-ee7afc76-1bb4-11e8-9c06-a19bd20c17fd.png;> ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user justinleet commented on the issue: https://github.com/apache/metron/pull/938 The Apache logo image used is a bit outdated. Could you swap it with the latest? Looks like it's at https://www.apache.org/foundation/press/kit/#links. I can only find it from the press links, but if anyone knows a better place to grab it from, we can pull from there. ---
[GitHub] metron issue #938: METRON-1457: Move ASF links to main page in the Metron we...
Github user anandsubbu commented on the issue: https://github.com/apache/metron/pull/938 Thank you @mmiklavc and @nickwallen for your reply on the dev list. Any feedback on this PR? ---
