Skip to content

Make sure multi-byte UTF8 characters are handled properly in data-latex attributes. (mathjax/MathJax#3575)#1505

Open
dpvc wants to merge 1 commit into
developfrom
issue3575
Open

Make sure multi-byte UTF8 characters are handled properly in data-latex attributes. (mathjax/MathJax#3575)#1505
dpvc wants to merge 1 commit into
developfrom
issue3575

Conversation

@dpvc

@dpvc dpvc commented Jun 13, 2026

Copy link
Copy Markdown
Member

This PR fixes a problem where unicode characters from planes other than Plane0 are not properly entered in some data-latex attributes. The problem is that for a character that requires multiple string character at the beginning of the attribute, we need to back up the correct number of characters (not just 1). The fix uses the string length of the unicode character passed to it.

Resolves issue mathjax/MathJax#3575.

@dpvc dpvc added this to the v4.1.3 milestone Jun 13, 2026
@dpvc dpvc requested a review from zorkow June 13, 2026 20:28
@codecov

codecov Bot commented Jun 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.47%. Comparing base (a65456c) to head (68caffc).

Additional details and impacted files
@@            Coverage Diff             @@
##           develop    #1505     +/-   ##
==========================================
  Coverage    86.47%   86.47%             
==========================================
  Files          340      340             
  Lines        86125    86125             
  Branches      3206     4848   +1642     
==========================================
  Hits         74476    74476             
+ Misses       11649    11628     -21     
- Partials         0       21     +21     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant