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

[GitBox <gi...@apache.org>]

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
   Abstract
   Prerequisites
   Tags
   Share this page buttons
   Video if available
   Content
   CTAs:
   * Related tutorials links
   * Share this page buttons
   
   With this PR I was testing a simple concept that uses Sphinx's templating system and the site's current layout as a precursor. 
   
   SK's advise was to redo the site to fix all of the bootstrap and Sphinx hacks so it is easy to update and will behave as expected. I won't attempt anything fancy after what I just went through to get even this working. 
   
   I can turn off the include for certain sections. I can add the text menu to the nav to align it with mobile. I can bump the res of the images. After that, it's pretty hard right now. 
   
   Also, I think the buttons should call some attention. We want people to click on them. I'd like to see more votes on limiting which pages they appear. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services