[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/10297 --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-165197885 LGTM. I'll merge this with master and branch-1.6 --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user parano commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47701271 --- Diff: docs/css/main.css --- @@ -171,24 +202,104 @@ a.anchorjs-link:hover { text-decoration: none; } * The left navigation bar. */ .left-menu-wrapper { - position: absolute; - height: 100%; - - width: 256px; - margin-top: -20px; - padding-top: 20px; + margin-left: 0px; + margin-right: 0px; background-color: #F0F8FC; + border-top-width: 0px; + border-left-width: 0px; + border-bottom-width: 0px; + margin-top: 0px; + width: 210px; + float: left; + position: absolute; } .left-menu { - position: fixed; - max-width: 350px; - - padding-right: 10px; - width: 256px; + padding: 0px; + width: 199px; } .left-menu h3 { margin-left: 10px; line-height: 30px; +} + +/** + * The collapsing button for the navigation bar. + */ +.nav-trigger { +position: fixed; +clip: rect(0, 0, 0, 0); +} + +.nav-trigger + label:after { +content: '»'; +} + +label { + z-index: 10; +} + +label[for="nav-trigger"] { +position: fixed; --- End diff -- I saw both 2 and 4 space indent in this file, it should be consistant --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user parano commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47702380 --- Diff: docs/css/main.css --- @@ -39,13 +39,26 @@ margin-left: 10px; } +/* body .container-wrapper { position: absolute; width: 100%; display: flex; } +*/ -body #content { +body .container-wrapper { + background-color: #FFF; + color: #1D1F22; + max-width: 1024px; + margin-top: 10px; + margin-left: auto; + margin-right: auto; + border-radius: 15px; + position: relative; +} + +body #contentREMOVE { --- End diff -- is that a typo? ```#contentREMOVE``` --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user parano commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47700990 --- Diff: docs/_layouts/global.html --- @@ -128,19 +128,31 @@ {% if page.url contains "/ml" %} {% include nav-left-wrapper-ml.html nav-mllib=site.data.menu-mllib nav-ml=site.data.menu-ml %} + + + --- End diff -- indent --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user parano commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47701840 --- Diff: docs/_layouts/global.html --- @@ -128,19 +128,31 @@ {% if page.url contains "/ml" %} {% include nav-left-wrapper-ml.html nav-mllib=site.data.menu-mllib nav-ml=site.data.menu-ml %} + --- End diff -- ```checked=""``` could be just ```checked``` --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user parano commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47703712 --- Diff: docs/css/main.css --- @@ -171,24 +202,104 @@ a.anchorjs-link:hover { text-decoration: none; } * The left navigation bar. */ .left-menu-wrapper { - position: absolute; - height: 100%; - - width: 256px; - margin-top: -20px; - padding-top: 20px; + margin-left: 0px; + margin-right: 0px; background-color: #F0F8FC; + border-top-width: 0px; + border-left-width: 0px; + border-bottom-width: 0px; + margin-top: 0px; + width: 210px; + float: left; + position: absolute; } .left-menu { - position: fixed; - max-width: 350px; - - padding-right: 10px; - width: 256px; + padding: 0px; + width: 199px; } .left-menu h3 { margin-left: 10px; line-height: 30px; +} + +/** + * The collapsing button for the navigation bar. + */ +.nav-trigger { +position: fixed; +clip: rect(0, 0, 0, 0); +} + +.nav-trigger + label:after { +content: '»'; +} + +label { + z-index: 10; +} + +label[for="nav-trigger"] { +position: fixed; +margin-left: 0px; +padding-top: 100px; +padding-left: 5px; +width: 10px; +height: 80%; +cursor: pointer; +background-size: contain; +background-color: #D4F0FF; +border-radius: 10px 0px 0px 10px; --- End diff -- the border-radius looks a bit wired since it's the only thing that has rounded corners in the page, try remove this line? --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user parano commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47702792 --- Diff: docs/css/main.css --- @@ -171,24 +202,104 @@ a.anchorjs-link:hover { text-decoration: none; } * The left navigation bar. */ .left-menu-wrapper { - position: absolute; - height: 100%; - - width: 256px; - margin-top: -20px; - padding-top: 20px; + margin-left: 0px; + margin-right: 0px; background-color: #F0F8FC; + border-top-width: 0px; + border-left-width: 0px; + border-bottom-width: 0px; + margin-top: 0px; + width: 210px; + float: left; + position: absolute; } .left-menu { - position: fixed; - max-width: 350px; - - padding-right: 10px; - width: 256px; + padding: 0px; + width: 199px; } .left-menu h3 { margin-left: 10px; line-height: 30px; +} + +/** + * The collapsing button for the navigation bar. + */ +.nav-trigger { +position: fixed; +clip: rect(0, 0, 0, 0); +} + +.nav-trigger + label:after { +content: '»'; +} + +label { --- End diff -- why not put this in the next selector ```label[for="nav-trigger"]``` --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47720994 --- Diff: docs/_layouts/global.html --- @@ -128,19 +128,31 @@ {% if page.url contains "/ml" %} {% include nav-left-wrapper-ml.html nav-mllib=site.data.menu-mllib nav-ml=site.data.menu-ml %} + + + --- End diff -- Fixed --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164948987 **[Test build #47764 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47764/consoleFull)** for PR 10297 at commit [`464dcb5`](https://github.com/apache/spark/commit/464dcb55c494d498e719c097bbac0d837d26e5ec). --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47721217 --- Diff: docs/css/main.css --- @@ -171,24 +202,104 @@ a.anchorjs-link:hover { text-decoration: none; } * The left navigation bar. */ .left-menu-wrapper { - position: absolute; - height: 100%; - - width: 256px; - margin-top: -20px; - padding-top: 20px; + margin-left: 0px; + margin-right: 0px; background-color: #F0F8FC; + border-top-width: 0px; + border-left-width: 0px; + border-bottom-width: 0px; + margin-top: 0px; + width: 210px; + float: left; + position: absolute; } .left-menu { - position: fixed; - max-width: 350px; - - padding-right: 10px; - width: 256px; + padding: 0px; + width: 199px; } .left-menu h3 { margin-left: 10px; line-height: 30px; +} + +/** + * The collapsing button for the navigation bar. + */ +.nav-trigger { +position: fixed; +clip: rect(0, 0, 0, 0); +} + +.nav-trigger + label:after { +content: '»'; +} + +label { + z-index: 10; +} + +label[for="nav-trigger"] { +position: fixed; --- End diff -- Fixed everywhere to 2 spaces --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47721302 --- Diff: docs/css/main.css --- @@ -39,13 +39,26 @@ margin-left: 10px; } +/* body .container-wrapper { position: absolute; width: 100%; display: flex; } +*/ -body #content { +body .container-wrapper { + background-color: #FFF; + color: #1D1F22; + max-width: 1024px; + margin-top: 10px; + margin-left: auto; + margin-right: auto; + border-radius: 15px; + position: relative; +} + +body #contentREMOVE { --- End diff -- Thanks! --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47721444 --- Diff: docs/css/main.css --- @@ -171,24 +202,104 @@ a.anchorjs-link:hover { text-decoration: none; } * The left navigation bar. */ .left-menu-wrapper { - position: absolute; - height: 100%; - - width: 256px; - margin-top: -20px; - padding-top: 20px; + margin-left: 0px; + margin-right: 0px; background-color: #F0F8FC; + border-top-width: 0px; + border-left-width: 0px; + border-bottom-width: 0px; + margin-top: 0px; + width: 210px; + float: left; + position: absolute; } .left-menu { - position: fixed; - max-width: 350px; - - padding-right: 10px; - width: 256px; + padding: 0px; + width: 199px; } .left-menu h3 { margin-left: 10px; line-height: 30px; +} + +/** + * The collapsing button for the navigation bar. + */ +.nav-trigger { +position: fixed; +clip: rect(0, 0, 0, 0); +} + +.nav-trigger + label:after { +content: '»'; +} + +label { + z-index: 10; +} + +label[for="nav-trigger"] { +position: fixed; +margin-left: 0px; +padding-top: 100px; +padding-left: 5px; +width: 10px; +height: 80%; +cursor: pointer; +background-size: contain; +background-color: #D4F0FF; +border-radius: 10px 0px 0px 10px; --- End diff -- Removing the border. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47721419 --- Diff: docs/css/main.css --- @@ -171,24 +202,104 @@ a.anchorjs-link:hover { text-decoration: none; } * The left navigation bar. */ .left-menu-wrapper { - position: absolute; - height: 100%; - - width: 256px; - margin-top: -20px; - padding-top: 20px; + margin-left: 0px; + margin-right: 0px; background-color: #F0F8FC; + border-top-width: 0px; + border-left-width: 0px; + border-bottom-width: 0px; + margin-top: 0px; + width: 210px; + float: left; + position: absolute; } .left-menu { - position: fixed; - max-width: 350px; - - padding-right: 10px; - width: 256px; + padding: 0px; + width: 199px; } .left-menu h3 { margin-left: 10px; line-height: 30px; +} + +/** + * The collapsing button for the navigation bar. + */ +.nav-trigger { +position: fixed; +clip: rect(0, 0, 0, 0); +} + +.nav-trigger + label:after { +content: '»'; +} + +label { --- End diff -- Good question. I tried to do this and it failed to take the z-index into account despite my best efforts. This is the only way I found to make it work. If you think it is too wide, then I can try to add a CSS class to 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164948293 Merged build finished. Test FAILed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164948295 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47762/ Test FAILed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47724053 --- Diff: docs/css/main.css --- @@ -101,6 +98,26 @@ a:hover code { max-width: 914px; } +.content { + z-index: 1; + position: relative; + background-color: #FFF; + max-width: 914px; + line-height: 1.6; /* Inspired by Github's wiki style */ + background-color: white; + padding-left: 15px; +} + +.content-with-sidebar { + z-index: 1; + position: relative; + background-color: #FFF; --- End diff -- ditto --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47724038 --- Diff: docs/css/main.css --- @@ -101,6 +98,26 @@ a:hover code { max-width: 914px; } +.content { + z-index: 1; + position: relative; + background-color: #FFF; --- End diff -- 2 instances of background-color here --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47727256 --- Diff: docs/css/main.css --- @@ -101,6 +98,26 @@ a:hover code { max-width: 914px; } +.content { + z-index: 1; + position: relative; + background-color: #FFF; + max-width: 914px; + line-height: 1.6; /* Inspired by Github's wiki style */ + background-color: white; + padding-left: 15px; +} + +.content-with-sidebar { + z-index: 1; + position: relative; + background-color: #FFF; --- End diff -- Done --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on a diff in the pull request: https://github.com/apache/spark/pull/10297#discussion_r47727246 --- Diff: docs/css/main.css --- @@ -101,6 +98,26 @@ a:hover code { max-width: 914px; } +.content { + z-index: 1; + position: relative; + background-color: #FFF; --- End diff -- oops thanks! --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164961668 **[Test build #47773 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47773/consoleFull)** for PR 10297 at commit [`19b92a2`](https://github.com/apache/spark/commit/19b92a21ecd157ad834cef81caf206cd26a84a45). --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user parano commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164907729 @thunterdb @jkbradley I left some comments on the css part, otherwise looks good to me --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164979002 **[Test build #47764 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47764/consoleFull)** for PR 10297 at commit [`464dcb5`](https://github.com/apache/spark/commit/464dcb55c494d498e719c097bbac0d837d26e5ec). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164979070 Merged build finished. Test PASSed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164979071 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47764/ Test PASSed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164986646 **[Test build #47773 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47773/consoleFull)** for PR 10297 at commit [`19b92a2`](https://github.com/apache/spark/commit/19b92a21ecd157ad834cef81caf206cd26a84a45). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164986946 Merged build finished. Test PASSed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164986950 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47773/ Test PASSed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164955741 Functionally, it seems good to me. (tested locally) --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164607406 **[Test build #2215 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2215/consoleFull)** for PR 10297 at commit [`b091b42`](https://github.com/apache/spark/commit/b091b42660ac510ba49ddf0274c779ae1c249b67). * This patch passes all tests. * This patch merges cleanly. * This patch adds no public classes. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
GitHub user thunterdb opened a pull request: https://github.com/apache/spark/pull/10297 [SPARK-12324][MLLIB][DOC] Fixes the sidebar in the ML documentation This fixes the sidebar, using a pure CSS mechanism to hide it when the browser's viewport is too narrow. Credit goes to the original author @Titan-C (mentioned in the NOTICE). Note that I am not a CSS expert, so I can only address comments up to some extent. Default view: https://cloud.githubusercontent.com/assets/7594753/11793597/6d1d6eda-a261-11e5-836b-6eb2054e9054.png;> When collapsed manually by the user: https://cloud.githubusercontent.com/assets/7594753/11793669/c991989e-a261-11e5-8bf6-aecf3bdb6319.png;> Disappears when column is too narrow: https://cloud.githubusercontent.com/assets/7594753/11793607/7754dbcc-a261-11e5-8b15-e0d074b0e47c.png;> Can still be opened by the user if necessary: https://cloud.githubusercontent.com/assets/7594753/11793612/7bf82968-a261-11e5-9cc3-e827a7a6b2b0.png;> You can merge this pull request into a Git repository by running: $ git pull https://github.com/thunterdb/spark 12324 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/spark/pull/10297.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 #10297 commit dc8d41b94aa49c31849596ea986368c3bb3c29e3 Author: Timothy HunterDate: 2015-12-14T19:39:54Z menu commit 94e82743956bce72f5ca2402be535fced36db24e Author: Timothy Hunter Date: 2015-12-14T20:00:50Z colors fixed commit b091b42660ac510ba49ddf0274c779ae1c249b67 Author: Timothy Hunter Date: 2015-12-14T20:41:28Z add the original author to the notice --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user thunterdb commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164556778 @jkbradley can you take a look at this fix? --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164560209 **[Test build #47677 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47677/consoleFull)** for PR 10297 at commit [`b091b42`](https://github.com/apache/spark/commit/b091b42660ac510ba49ddf0274c779ae1c249b67). --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164586220 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47677/ Test FAILed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164586214 Merged build finished. Test FAILed. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164586030 **[Test build #47677 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/47677/consoleFull)** for PR 10297 at commit [`b091b42`](https://github.com/apache/spark/commit/b091b42660ac510ba49ddf0274c779ae1c249b67). * This patch **fails SparkR unit tests**. * This patch merges cleanly. * This patch adds no public classes. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164588869 **[Test build #2215 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2215/consoleFull)** for PR 10297 at commit [`b091b42`](https://github.com/apache/spark/commit/b091b42660ac510ba49ddf0274c779ae1c249b67). --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
[GitHub] spark pull request: [SPARK-12324][MLLIB][DOC] Fixes the sidebar in...
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/10297#issuecomment-164588847 Tested locally. Functionally, this seems fine. I'll rely on someone else to check the CSS. --- 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. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org