Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Jan 8, 2021

What changes were proposed in this pull request?

This PR tweaks the width of left-menu of Spark SQL Guide.
When I view the Spark SQL Guide with browsers on macOS, the title Spark SQL Guide looks prettily.
But I often use Pop!_OS, an Ubuntu variant, and the title is overlapped with browsers on it.
spark-sql-guide-layout-before

After this change, the title is no longer overlapped.
spark-sql-guide-layout-after

Why are the changes needed?

For the pretty layout.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Built the document with cd docs && SKIP_API=1 jekyll build and confirmed the layout.

@SparkQA
Copy link

SparkQA commented Jan 8, 2021

Test build #133834 has finished for PR 31091 at commit 2b08e31.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Jan 8, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38423/

@github-actions github-actions bot added the DOCS label Jan 8, 2021
@SparkQA
Copy link

SparkQA commented Jan 8, 2021

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38423/

@srowen
Copy link
Member

srowen commented Jan 8, 2021

I'm going to merge to master, and 3.1 as well, because the original JIRA was resolved for 3.1 and this is so minor.

@srowen srowen closed this in 023eba2 Jan 8, 2021
srowen pushed a commit that referenced this pull request Jan 8, 2021
…QL Guide

### What changes were proposed in this pull request?

This PR tweaks the width of left-menu of Spark SQL Guide.
When I view the Spark SQL Guide with browsers on macOS, the title `Spark SQL Guide` looks prettily.
But I often use Pop!_OS, an Ubuntu variant, and the title is overlapped with browsers on it.
![spark-sql-guide-layout-before](https://user-images.githubusercontent.com/4736016/104002743-d56cc200-51e4-11eb-9e3a-28abcd46e0bf.png)

After this change, the title is no longer overlapped.
![spark-sql-guide-layout-after](https://user-images.githubusercontent.com/4736016/104002847-f9c89e80-51e4-11eb-85c0-01d69cee46b7.png)

### Why are the changes needed?

For the pretty layout.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Built the document with `cd docs && SKIP_API=1 jekyll build` and confirmed the layout.

Closes #31091 from sarutak/modify-layout-sparksql-guide.

Authored-by: Kousuke Saruta <[email protected]>
Signed-off-by: Sean Owen <[email protected]>
(cherry picked from commit 023eba2)
Signed-off-by: Sean Owen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants