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-46151][PYTHON][DOCS] Hide the More drop-down button in the PySpark docs navigation bar #44068

Closed
wants to merge 2 commits into from

Conversation

panbingkun
Copy link
Contributor

What changes were proposed in this pull request?

The pr aims to hide the More drop-down button in the PySpark docs navigation bar.

Why are the changes needed?

Because there is only one item under the More drop-down button, as following:
image

Obviously, in our scenario, eliminating More drop-down button and directly displaying Migration Guides menu in the navigation bar is more intuitive and convenient.

After:
image

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Manually test.

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

No.

@panbingkun
Copy link
Contributor Author

panbingkun commented Nov 29, 2023

cc @HyukjinKwon @itholic

@@ -196,6 +196,7 @@
# further. For a list of options available for each theme, see the
# documentation.
html_theme_options = {
"header_links_before_dropdown": 6,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

default value is 5

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, I was also thinking of fixing this.

Copy link
Contributor

@itholic itholic left a comment

Choose a reason for hiding this comment

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

LGTM.

@itholic
Copy link
Contributor

itholic commented Nov 29, 2023

Btw, seems like the code-block style is not applied properly in the PR title?

Screenshot 2023-11-29 at 3 16 42 PM

Could you address it?

@panbingkun panbingkun changed the title [SPARK-46151][PYTHON][DOCS] Hide the More" drop-down button in the PySpark docs navigation bar [SPARK-46151][PYTHON][DOCS] Hide the More drop-down button in the PySpark docs navigation bar Nov 29, 2023
@panbingkun
Copy link
Contributor Author

Btw, seems like the code-block style is not applied properly in the PR title?

Screenshot 2023-11-29 at 3 16 42 PM Could you address it?

Done, is it OK now?

@itholic
Copy link
Contributor

itholic commented Nov 29, 2023

I guess maybe you wanted to apply the style something like:

Hide the "More" drop-down button in the PySpark docs navigation bar

or

Hide the "More" drop-down button in the PySpark docs navigation bar

But it's not a big deal, so the PR looks still good to me :-)

@HyukjinKwon
Copy link
Member

Merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants