Skip to content

Conversation

@SirLouen
Copy link
Member

@SirLouen SirLouen commented Nov 2, 2025

What?

Closes #72899

The Math component should not have Edit as HTML, as the whole Component is managed through attributes. Changes in the HTML won't reflect on the component. Also, little improvements as we can already destructure latex, we can also destructure mathML

Why?

According to #72899, while editing as HTML, there is a visual glitch on the textarea. The solution could be removing such textarea, but in reality the true solution is completely removing the HTML editing feature.

How?

Removing the supports

Before After
image image

@github-actions
Copy link

github-actions bot commented Nov 2, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: SirLouen <[email protected]>
Co-authored-by: t-hamano <[email protected]>
Co-authored-by: youknowriad <[email protected]>
Co-authored-by: ntsekouras <[email protected]>
Co-authored-by: shimotmk <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@SirLouen SirLouen requested review from ntsekouras and removed request for DAreRodz, ZebulanStanphill, fabiankaegy and tellthemachines November 4, 2025 17:55
@SirLouen SirLouen force-pushed the patch/72899 branch 5 times, most recently from a85f0ae to a65ceb5 Compare November 4, 2025 23:40
Copy link
Contributor

@t-hamano t-hamano left a comment

Choose a reason for hiding this comment

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

LGTM! Based on my thoughts here, I would like to approve this PR. In the future, if it becomes possible to convert MathML to LaTeX, let's consider reinstating HTML editing at that time.

@t-hamano t-hamano added the Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 5, 2025
@t-hamano t-hamano merged commit e491dc1 into WordPress:trunk Nov 5, 2025
46 of 47 checks passed
@github-actions github-actions bot added this to the Gutenberg 22.1 milestone Nov 5, 2025
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

There was a conflict while trying to cherry-pick the commit to the wp/6.9 branch. Please resolve the conflict manually and create a PR to the wp/6.9 branch.

PRs to wp/6.9 are similar to PRs to trunk, but you should base your PR on the wp/6.9 branch instead of trunk.

# Checkout the wp/6.9 branch instead of trunk.
git checkout wp/6.9

# Create a new branch for your PR.
git checkout -b my-branch

# Cherry-pick the commit.
git cherry-pick e491dc1492f4105be54949c7a16dc011daee267a

# Check which files have conflicts.
git status

# Resolve the conflict...
# Add the resolved files to the staging area.
git status
git add .
git cherry-pick --continue

# Push the branch to the repository
git push origin my-branch

# Create a PR and set the base to the wp/6.9 branch.
# See https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-base-branch-of-a-pull-request.

@t-hamano
Copy link
Contributor

t-hamano commented Nov 5, 2025

@SirLouen, the automatic cherry-pick to the wp/6.9 branch failed, so we need to do a manual cherry-pick. Would you like to try it?

See: #72910 (comment)

SirLouen added a commit to SirLouen/gutenberg that referenced this pull request Nov 5, 2025
* Math: Fixing Edit as HTML feature and code editor sync

* fix: More syntantical uniformity

* fix: Add support indication for core/math block in documentation

Co-authored-by: SirLouen <[email protected]>
Co-authored-by: t-hamano <[email protected]>
Co-authored-by: youknowriad <[email protected]>
Co-authored-by: ntsekouras <[email protected]>
Co-authored-by: shimotmk <[email protected]>
SirLouen added a commit to SirLouen/gutenberg that referenced this pull request Nov 5, 2025
* Math: Fixing Edit as HTML feature and code editor sync

* fix: More syntantical uniformity

* fix: Add support indication for core/math block in documentation

Co-authored-by: SirLouen <[email protected]>
Co-authored-by: t-hamano <[email protected]>
Co-authored-by: youknowriad <[email protected]>
Co-authored-by: ntsekouras <[email protected]>
Co-authored-by: shimotmk <[email protected]>
@SirLouen SirLouen deleted the patch/72899 branch November 5, 2025 11:22
@SirLouen
Copy link
Member Author

SirLouen commented Nov 5, 2025

Ok I sorted it #73003

t-hamano added a commit that referenced this pull request Nov 5, 2025
* Math: Fixing Edit as HTML feature and code editor sync

* fix: More syntantical uniformity

* fix: Add support indication for core/math block in documentation

Co-authored-by: SirLouen <[email protected]>
Co-authored-by: t-hamano <[email protected]>
Co-authored-by: youknowriad <[email protected]>
Co-authored-by: ntsekouras <[email protected]>
Co-authored-by: shimotmk <[email protected]>
@t-hamano
Copy link
Contributor

t-hamano commented Nov 5, 2025

This pull request was manually cherry-picked into the wp/6.9 branch: #73003

@t-hamano t-hamano added Backported to WP Core Pull request that has been successfully merged into WP Core and removed Backport to WP 6.9 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta labels Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backported to WP Core Pull request that has been successfully merged into WP Core [Block] Math [Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Math: When you select "Edit as HTML," the text area moves to the upper left corner of the editor.

2 participants