You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by Leemoonsoo <gi...@git.apache.org> on 2015/11/22 09:25:58 UTC

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

GitHub user Leemoonsoo opened a pull request:

    https://github.com/apache/incubator-zeppelin/pull/461

    ZEPPELIN-445 Improve documentation menu

    https://issues.apache.org/jira/browse/ZEPPELIN-445
    
    This PR improve menu of documentation.
    
    Before
    ![image](https://cloud.githubusercontent.com/assets/1540981/11322744/65f08e06-913d-11e5-86b0-773bbc68b574.png)
    
    After
    ![image](https://cloud.githubusercontent.com/assets/1540981/11322750/ef14ee8e-913d-11e5-989c-1e54f61c1b07.png)
    
    ![image](https://cloud.githubusercontent.com/assets/1540981/11322753/f76fd60c-913d-11e5-955d-4c7cd0f78300.png)
    
    ![image](https://cloud.githubusercontent.com/assets/1540981/11322754/fe2344de-913d-11e5-94fd-58d2f718775e.png)
    
    ![image](https://cloud.githubusercontent.com/assets/1540981/11322756/0476952a-913e-11e5-8be5-c9fcb32498b0.png)
    
    


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Leemoonsoo/incubator-zeppelin ZEPPELIN-445

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-zeppelin/pull/461.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #461
    
----
commit 8bc03e5d039ab8e2dfee1b5fc5d5d79607515742
Author: Lee moon soo <mo...@apache.org>
Date:   2015-11-22T08:20:02Z

    Improve menu structure

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by jeffsteinmetz <gi...@git.apache.org>.
Github user jeffsteinmetz commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-158778099
  
    We could also reference the Vagrant virtual machine in this pr under Quickstart or More:   https://github.com/apache/incubator-zeppelin/pull/460


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-158738875
  
    With ZEPPELIN-412, website is now maintained in gh-pages branch. /docs dir in master branch is only taking care of documentation for  individual version.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by bzz <gi...@git.apache.org>.
Github user bzz commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-158738710
  
    Tested, menu structure looks really great! 
    
    May be I'm confusing something but in **local build** I can see
    
    ![screen shot 2015-11-22 at 18 09 19](https://cloud.githubusercontent.com/assets/5582506/11322924/4f3dcb2c-9144-11e5-8723-60d400b6b24e.png)
    
    While **current website** looks like
    
    ![screen shot 2015-11-22 at 18 09 38](https://cloud.githubusercontent.com/assets/5582506/11322925/573b8aa8-9144-11e5-956a-dcfc7c62ddb6.png)
    
    Am I doing something wrong or do we want to lose Jumbotrone with video link as well as other menus, like "Apache", Download, Community?
    
    A nit probably, but if some HTML ninja could help with 
     - vertical alignment of the version label in header with other menus
     - color\behavior of the triangles on the dropdowns (white by default, blue when clicked and point up)
    
    that would make things a bit more polished. 


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-158851297
  
    cool!


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-159351559
  
    @alex Changed the caret(triangles) color. 
    
    Could't do 'vertical alignment of the version label in header with other menus', while it looks bit more weird when i trying it. I think it would be better someone who have better expertise in design handle this.
    
    Now menu looks like
    ![image](https://cloud.githubusercontent.com/assets/1540981/11375242/7a5a2f32-931d-11e5-9c30-ad86e05b90e8.png)
    
    Ready to merge.



---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-zeppelin/pull/461


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-158738329
  
    How Cool !!! 


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by corneadoug <gi...@git.apache.org>.
Github user corneadoug commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-159464439
  
    @Leemoonsoo Create an issue for the vertical align, and I will fix it


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-zeppelin pull request: ZEPPELIN-445 Improve documentatio...

Posted by Leemoonsoo <gi...@git.apache.org>.
Github user Leemoonsoo commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/461#issuecomment-159552009
  
    @corneadoug Thanks. I have created issue https://issues.apache.org/jira/browse/ZEPPELIN-435
    Then, I'm merging this PR.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---