docs(importmaps): use importmaps for external script

[daker]

Context

Recommend the use of importmaps

Results

Changes

• Documentation and TypeScript definitions were updated to match those changes

PR and Code Checklist

• semantic-release commit messages
• Run npm run reformat to have correctly formatted code

Testing

• This change adds or fixes unit tests
• Tested environment:
  • vtk.js:
  • OS:
  • Browser:

[finetjul]

There are other places that reference https://unpkg.com/vtk.js, you might want to modify those as well...

@floryst should we mention both options ? with and without import maps ?

[finetjul]

LGTM

[floryst]

Overall looks good. I would like to request that we add the ES module code in addition to the UMD bundle. We can have two headings, such as "Using ES Modules" and "Using the UMD bundle", in that order.

[daker]

@floryst when you say two headings, you mean one for import * from '@kitware/...' and one for vtk.Rendering ?

[floryst]

Yes. I see you replaced the UMD code with the importmaps code. While I agree with moving forward with importmaps (esp given its support across major browsers), I would like to keep some of the UMD docs around for a bit longer.

[floryst]

In revisiting this, I think we can go ahead and drop the UMD instructions. Modern baseline browsers support importmaps, and if we really need to support much older browsers, something else will likely break.

[sankhesh]

@floryst @daker Is this ready to be merged?

[daker]

@sankhesh for me it's ok