fix: correct extension marker dimensions for mobile/iOS rendering#7538
fix: correct extension marker dimensions for mobile/iOS rendering#7538cyphercodes wants to merge 1 commit intomermaid-js:developfrom
Conversation
|
✅ Deploy Preview for mermaid-js ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
@mermaid-js/examples
mermaid
@mermaid-js/layout-elk
@mermaid-js/layout-tidy-tree
@mermaid-js/mermaid-zenuml
@mermaid-js/parser
@mermaid-js/tiny
commit: |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7538 +/- ##
==========================================
- Coverage 3.34% 3.34% -0.01%
==========================================
Files 524 525 +1
Lines 55256 55267 +11
Branches 795 795
==========================================
Hits 1850 1850
- Misses 53406 53417 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
Issue
Fixes #7518
Problem
The extensionStart marker (used for implementation/realization arrows in class diagrams) had incorrect dimensions:
These typos caused the arrow markers to render incorrectly or not at all on mobile devices and iOS Safari.
Solution
Corrected the marker dimensions to match the extensionEnd marker and the margin variants:
Verification
This fix aligns the extensionStart marker with:
The path data (M 1,7 L18,13 V 1 Z) expects a viewBox of approximately 20x14, so markerWidth=20 and markerHeight=28 are appropriate values.