Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-44986][DOCS] There should be a gap at the bottom of the HTML #42702

Closed
wants to merge 4 commits into from

Conversation

panbingkun
Copy link
Contributor

@panbingkun panbingkun commented Aug 28, 2023

What changes were proposed in this pull request?

The pr aims to add gap at the bottom of the HTML.

Why are the changes needed?

The old document style has good white space at the bottom, but the latest document has lost this piece, which looks unattractive and borderless.
image

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manual testing.

SKIP_API=1 bundle exec jekyll serve --watch

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added the DOCS label Aug 28, 2023
@panbingkun
Copy link
Contributor Author

Before:
image

After:
image

@zhengruifeng
Copy link
Contributor

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

Please don't mix 2.align example order(Python -> Scala/Java -> R) in this PR.

Copy link
Contributor

@allisonwang-db allisonwang-db left a comment

Choose a reason for hiding this comment

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

Thanks for the adjustment! Let's also update the PR title to include the example order updates.

@@ -470,19 +470,19 @@ Congratulations on running your first Spark application!
* For an in-depth overview of the API, start with the [RDD programming guide](rdd-programming-guide.html) and the [SQL programming guide](sql-programming-guide.html), or see "Programming Guides" menu for other components.
* For running applications on a cluster, head to the [deployment overview](cluster-overview.html).
* Finally, Spark includes several samples in the `examples` directory
([Scala]({{site.SPARK_GITHUB_URL}}/tree/master/examples/src/main/scala/org/apache/spark/examples),
([Python]({{site.SPARK_GITHUB_URL}}/tree/master/examples/src/main/python),
Copy link
Contributor

Choose a reason for hiding this comment

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

Are there any other places where the Python one is not the first example?

@zhengruifeng
Copy link
Contributor

zhengruifeng commented Aug 29, 2023

Please don't mix 2.align example order(Python -> Scala/Java -> R) in this PR.

+1, it is irrelevant. Should open a separate PR if needed

@panbingkun
Copy link
Contributor Author

panbingkun commented Aug 29, 2023

Okay, Let me split it into a new separate PR, because I found similar order issues on other pages.
A new separate PR: #42712

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM. Thank you, @panbingkun and all.

@dongjoon-hyun
Copy link
Member

Merged to master for Apache Spark 4.0.0.

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.

4 participants