-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Math: Fixing Edit as HTML feature and code editor sync #72910
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
Conversation
|
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 If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
a85f0ae to
a65ceb5
Compare
t-hamano
left a comment
There was a problem hiding this 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.
|
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. |
|
@SirLouen, the automatic cherry-pick to the See: #72910 (comment) |
* 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]>
* 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]>
|
Ok I sorted it #73003 |
* 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]>
|
This pull request was manually cherry-picked into the |
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 destructuremathMLWhy?
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