[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-12-21 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-449520266 Closing this PR. I can try another round of adding social buttons later. The other updates on the community page are already in #13705.

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-10-30 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-434494943 @ankkhedia Yes, the preview link is updated and the images are all there. This i

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-10-19 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-431509221 Looks fine... Travis just timed out. So this ready for review/approval/merge. Th

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-10-19 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-431464459 @Roshrini Ok, resolved conflicts... let's see if it clears CI. This is an automa

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-09-26 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-424815178 Since ApacheCon is almost over, I should remove that logo, and rebase all of this to make sure it's all working as expected. That would take care

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-09-04 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-418471260 @lupesko - do you still request changes? The latest is hosted here: http://34.201.8.176/versions/social_media_update_v2/ ---

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-28 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-416817078 @nswamy - Do you still request changes? This is an automated message from the Ap

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-27 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-416393108 Ok, here's the latest: http://34.201.8.176/versions/social_media_update_v2/ I tested on a high res monitor and fixed the padding so ther

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-23 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-415509287 @nswamy This one uses the logos from the platforms rather than trying to conform them to the mxnet blue background. http://34.201.8.176/versio

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-22 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-415236474 @simoncorstonoliver Thanks for your feedback. I only include the buttons on the home page now. I've updated the Youtube link. --

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-16 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-413600077 @lebeg Take a look at the [preview on your mobile](http://34.201.8.176/versions/social_media_update/) or just resize the screen to pretty small

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-09 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-411858390 @nswamy I fixed the responsive design for your higher res monitor. Though I'm sure the css could use an overhaul. -

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-09 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-411823586 I'm using svg images instead now, so things should look good an whatever resolution, plus they're small. And yes, I put them in the mxnet repo be

[GitHub] aaronmarkham commented on issue #12102: site-wide social include

2018-08-09 Thread GitBox
aaronmarkham commented on issue #12102: site-wide social include URL: https://github.com/apache/incubator-mxnet/pull/12102#issuecomment-411778580 @swamy for the tutorial pages I had something else in mind. I would provide a different layout. It would stack like... Nav Title Abst