[GitHub] zeppelin issue #2394: [gh-pages] [ZEPPELIN-2597] adding technologies and dep...

[AhyoungRyu]

Github user AhyoungRyu commented on the issue:

    https://github.com/apache/zeppelin/pull/2394
  
    Thanks for the great work @soralee! And also thanks for detailed review 
@1ambda. 
    Tested and looks nice! 
    I really like the most of parts. Some minor suggestion is
    
    #### Technologies
     - how about using same border color with "Deployment" box `#999` instead 
of `#4881B3`?
    <img width="997" alt="screen shot 2017-06-06 at 7 36 05 pm" 
src="https://user-images.githubusercontent.com/10060731/26859880-7f3df818-4aef-11e7-9364-a0274c2642bf.png";>
    
    #### Deployment
     -  how about putting "LEARN MORE" buttons in each deployment section box 
in the middle instead of right side? 
     - and also we can remove the single & multi user font awesome icons since 
the text ("single" and "multi") itself already explain what "single-user" and 
"multi-user" are meaning for.
    
    How about this? It's just my opinion. Please have a look and feel free to 
share your thought :)
    
    <img width="990" alt="screen shot 2017-06-06 at 7 25 45 pm" 
src="https://user-images.githubusercontent.com/10060731/26859800-0d9d5cf8-4aef-11e7-804c-3f93a1ae035d.png";>
    
    
    
    



---
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.
---