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-49072][DOCS] Fix abnormal display of text content which contains two $ in one line but non-formula in docs #47548

Closed
wants to merge 1 commit into from

Conversation

wayneguow
Copy link
Contributor

What changes were proposed in this pull request?

There are some display exceptions in some documents currently, for examples:

The reason is that the MathJax javascript package will display the content between two $ as a formula.

This PR aims to fix abnormal display of text content which contains two $ in one line but not non-formula in docs.

Why are the changes needed?

Fix doc display exceptions.

Does this PR introduce any user-facing change?

Yes, Improve user experience about docs.

How was this patch tested?

Local manual tests with command SKIP_API=1 bundle exec jekyll build --watch.

The new results after this PR:
image

image

image

image

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

No.

@wayneguow wayneguow changed the title [SPARK-49072][DOCS] Fix abnormal display of text content which contains two $ in one line but not non-formula in docs [SPARK-49072][DOCS] Fix abnormal display of text content which contains two $ in one line but non-formula in docs Jul 31, 2024
@wayneguow
Copy link
Contributor Author

cc @HyukjinKwon @allisonwang-db

@github-actions github-actions bot added the DOCS label Jul 31, 2024
@HyukjinKwon
Copy link
Member

Merged to master.

fusheng9399 pushed a commit to fusheng9399/spark that referenced this pull request Aug 6, 2024
…ns two $ in one line but non-formula in docs

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

There are some display exceptions in some documents currently, for examples:

- https://spark.apache.org/docs/3.5.1/running-on-kubernetes.html#secret-management
![image](https://github.com/user-attachments/assets/5a4fa4e0-b773-4007-96d0-c036bc7e0c13)

- https://spark.apache.org/docs/latest/sql-migration-guide.html
![image](https://github.com/user-attachments/assets/e5f7ea17-9573-4917-b9cd-e36fd83d35fb)

The reason is that the `MathJax` javascript package will display the content between two $ as a formula.

This PR aims to fix abnormal display of text content which contains two $ in one line but not non-formula in docs.
### Why are the changes needed?

Fix doc display exceptions.

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

Yes, Improve user experience about docs.

### How was this patch tested?

Local manual tests with command `SKIP_API=1 bundle exec jekyll build --watch`.

The new results after this PR:
![image](https://github.com/user-attachments/assets/3d0b62fc-44da-45cd-a295-5d098ce3b8ec)

![image](https://github.com/user-attachments/assets/f8884a26-7029-4926-a290-a24b7ae75fa4)

![image](https://github.com/user-attachments/assets/6d7a1289-b459-4ad3-8636-e4713ead3921)

![image](https://github.com/user-attachments/assets/216dbd8d-4bd5-43b6-abc5-082aea543888)

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

No.

Closes apache#47548 from wayneguow/latex_error.

Authored-by: Wei Guo <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
attilapiros pushed a commit to attilapiros/spark that referenced this pull request Oct 4, 2024
…ns two $ in one line but non-formula in docs

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

There are some display exceptions in some documents currently, for examples:

- https://spark.apache.org/docs/3.5.1/running-on-kubernetes.html#secret-management
![image](https://github.com/user-attachments/assets/5a4fa4e0-b773-4007-96d0-c036bc7e0c13)

- https://spark.apache.org/docs/latest/sql-migration-guide.html
![image](https://github.com/user-attachments/assets/e5f7ea17-9573-4917-b9cd-e36fd83d35fb)

The reason is that the `MathJax` javascript package will display the content between two $ as a formula.

This PR aims to fix abnormal display of text content which contains two $ in one line but not non-formula in docs.
### Why are the changes needed?

Fix doc display exceptions.

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

Yes, Improve user experience about docs.

### How was this patch tested?

Local manual tests with command `SKIP_API=1 bundle exec jekyll build --watch`.

The new results after this PR:
![image](https://github.com/user-attachments/assets/3d0b62fc-44da-45cd-a295-5d098ce3b8ec)

![image](https://github.com/user-attachments/assets/f8884a26-7029-4926-a290-a24b7ae75fa4)

![image](https://github.com/user-attachments/assets/6d7a1289-b459-4ad3-8636-e4713ead3921)

![image](https://github.com/user-attachments/assets/216dbd8d-4bd5-43b6-abc5-082aea543888)

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

No.

Closes apache#47548 from wayneguow/latex_error.

Authored-by: Wei Guo <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
himadripal pushed a commit to himadripal/spark that referenced this pull request Oct 19, 2024
…ns two $ in one line but non-formula in docs

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

There are some display exceptions in some documents currently, for examples:

- https://spark.apache.org/docs/3.5.1/running-on-kubernetes.html#secret-management
![image](https://github.com/user-attachments/assets/5a4fa4e0-b773-4007-96d0-c036bc7e0c13)

- https://spark.apache.org/docs/latest/sql-migration-guide.html
![image](https://github.com/user-attachments/assets/e5f7ea17-9573-4917-b9cd-e36fd83d35fb)

The reason is that the `MathJax` javascript package will display the content between two $ as a formula.

This PR aims to fix abnormal display of text content which contains two $ in one line but not non-formula in docs.
### Why are the changes needed?

Fix doc display exceptions.

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

Yes, Improve user experience about docs.

### How was this patch tested?

Local manual tests with command `SKIP_API=1 bundle exec jekyll build --watch`.

The new results after this PR:
![image](https://github.com/user-attachments/assets/3d0b62fc-44da-45cd-a295-5d098ce3b8ec)

![image](https://github.com/user-attachments/assets/f8884a26-7029-4926-a290-a24b7ae75fa4)

![image](https://github.com/user-attachments/assets/6d7a1289-b459-4ad3-8636-e4713ead3921)

![image](https://github.com/user-attachments/assets/216dbd8d-4bd5-43b6-abc5-082aea543888)

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

No.

Closes apache#47548 from wayneguow/latex_error.

Authored-by: Wei Guo <[email protected]>
Signed-off-by: Hyukjin Kwon <[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.

2 participants