Pending improvements - Tracker

[sidharthv96]

Noting down possible improvements while working on #3336

• Set viewport size in cypress config and remove 31 occurrences of cy.viewport(1440, 1024); from tests.
• Figure out if tests can be made faster
• Fix usage of arrowMarkerAbsolute
  • Still broken for flowchart v1
  • Fixed for flowchart-v2 in 595f768 (although it's a bit of a hack)
• Make tsconfig stricter ("noImplicitAny": true, etc)
• Add TS specific eslint configs
• Switch from moment to luxon as moment is deprecated. #3385
• htmlLabels should be moved up from flowchart to config. #3337
• Remove rogue console.log #3339
• Remove flowchart as default fallback in detectType. #3391
• Add a config validator and organise global configs #3397
• DT: Remove mermaid type definitions DefinitelyTyped/DefinitelyTyped#63319
• Improve init function and how selector and nodes are passed. #3394
• Run e2e test in parallel batches #3494

[dwhelan]

Hey @sidharthv96! I have a fix for #1871 where markers spilled over with multiple diagrams on the same html page. The fix was to create unique id's for each marker using the id in the svg element as a prefix. So, the id of pointEnd would now be something like mermaid-123456789-pointEnd.

I am looking for related issues because I suspect my fix could resolve other oddities with rendering markers. So I was wondering what the purpose of arrowMarkerAbsolute is. Is it possible that we might not need absolute markers if they were introduced to solve something that unique marker ids would address?