[GitHub] zeppelin issue #2381: [WIP] [gh-pages] [ZEPPELIN-2597] Updating offcial main...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

    https://github.com/apache/zeppelin/pull/2381
  
    Thank you for great work! Especially big +1 for adding this section
    
    <img width="1219" alt="screen shot 2017-06-04 at 11 27 48 am" 
src="https://cloud.githubusercontent.com/assets/10060731/26764287/e5b90aa2-4918-11e7-908c-1fdfd4870037.png";>
    
    I've always thought we need this kind of guide in first page of website.
    
    I quickly look through the change and here are some thought in my head. 
    
    ### 1. Navbar
     - before 
    <img width="1144" alt="screen shot 2017-06-04 at 11 20 39 am" 
src="https://cloud.githubusercontent.com/assets/10060731/26764234/1be270e2-4918-11e7-976c-ed07289479b2.png";>
    
     - after 
    <img width="1185" alt="screen shot 2017-06-04 at 11 20 22 am" 
src="https://cloud.githubusercontent.com/assets/10060731/26764235/1ec0bd82-4918-11e7-981c-4ba4f3c119b8.png";>
    
    Adding "Quick Start" menu and switching menu order are really good idea I 
think. But not sure about making the font size bigger and adding too much 
shadow. So in my opinion, just below work will be enough.
    
        - Adding "Quick Start" menu
        - Switching menu order 
    
    
    ### 2. Main
    I also agree this part doesn't have really good looking.
    <img width="1217" alt="screen shot 2017-06-04 at 11 37 38 am" 
src="https://cloud.githubusercontent.com/assets/10060731/26764349/4a00b96e-491a-11e7-9f06-6841bff11b50.png";>
    
    But not sure the new change is definitely better than current one.
    <img width="1174" alt="screen shot 2017-06-04 at 11 38 22 am" 
src="https://cloud.githubusercontent.com/assets/10060731/26764350/55ed2a46-491a-11e7-8f14-12cd3ff2385b.png";>
    
    
    
    I think it'll be better to update other parts expect "Technologies" & 
"Deployments" that you newly added in another PR step by step. That would be 
easier to get review/ consensus from other ppl and also easier to make it 
merged. I mean we can separate the work like below. 
    
    ```
    1. update navbar
    2. update main section
    3. adding "Technologies" & "Deployment" section
    4. minor styling 
    ....
    ```
    What do you think? 
    Thanks again for staring this work. 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---