Skip to content

Conversation

@gengliangwang
Copy link
Member

@gengliangwang gengliangwang commented Dec 15, 2020

What changes were proposed in this pull request?

If the text in the left menu of Spark is too long, it will be hidden.
sql1

This PR is to fix the style issue.

Why are the changes needed?

Improve the UI of Spark documentation.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Manual test
After changes:
sql2

@github-actions github-actions bot added the DOCS label Dec 15, 2020
@gengliangwang gengliangwang changed the title [SPARK-33796][Doc] Show hidden text from the left menu of Spark Doc [SPARK-33796][DOC] Show hidden text from the left menu of Spark Doc Dec 15, 2020
@gengliangwang gengliangwang changed the title [SPARK-33796][DOC] Show hidden text from the left menu of Spark Doc [SPARK-33796][DOCS] Show hidden text from the left menu of Spark Doc Dec 15, 2020
}

.left-menu {
width: 399px;
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The width is specified as 210px in its parent .left-menu-wrapper

@SparkQA
Copy link

SparkQA commented Dec 15, 2020

Test build #132838 has finished for PR 30786 at commit 946e0a8.

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

@SparkQA
Copy link

SparkQA commented Dec 15, 2020

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

@SparkQA
Copy link

SparkQA commented Dec 15, 2020

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

@HyukjinKwon
Copy link
Member

Thanks @gengliangwang for fixing this!

@HyukjinKwon
Copy link
Member

HyukjinKwon commented Dec 16, 2020

Merged to master and branch-3.1.

HyukjinKwon pushed a commit that referenced this pull request Dec 16, 2020
### What changes were proposed in this pull request?

If the text in the left menu of Spark is too long, it will be hidden.
![sql1](https://user-images.githubusercontent.com/1097932/102249583-5ae7a580-3eb7-11eb-813c-f2e2fe019d28.jpeg)

This PR is to fix the style issue.

### Why are the changes needed?

Improve the UI of Spark documentation.

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

No
### How was this patch tested?

Manual test
After changes:
![sql2](https://user-images.githubusercontent.com/1097932/102249603-5fac5980-3eb7-11eb-806d-4e7b8248e6b6.jpeg)

Closes #30786 from gengliangwang/fixDocStyle.

Authored-by: Gengliang Wang <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit dd042f5)
Signed-off-by: HyukjinKwon <[email protected]>
@HyukjinKwon
Copy link
Member

@gengliangwang, should we fix it in branch-3.0 too? it has a conflict.

@gengliangwang
Copy link
Member Author

@HyukjinKwon sure, I will do it.

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