Closed (fixed)
Project:
Importmaps
Version:
1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Nov 2023 at 22:33 UTC
Updated:
21 Nov 2023 at 22:44 UTC
Jump to comment: Most recent
It looks like we had the same idea around the same time: https://www.drupal.org/project/importmap
(I think your module did not exist yet when I looked for existing implementations a few weeks ago.)
Your approach seems definitely cleaner.
I am totally OK with archiving my module and redirecting to yours, but they would need to have feature parity before doing this.
Here is what I think my module does that yours does not yet:
scopes key in the importmap JSON)<link rel="modulepreload"> tags for modules in the importmap (this can help the browser parallelize requests, especially with long import chains)
Comments
Comment #2
larowlanThanks for listing the differences.
FWIW
, there is a standard alter hook here courtesy of the plugin manager so that is supported.
But you're correct, the other three aren't supported.
I think we'd be better to try and get this into core by 10.3 and mark both issues as deprecated?
I'm not using this module in production anywhere (and there is no tagged release). So I think I'll deprecate it now and update the project page to point to yours and the core issue.
Comment #3
larowlanComment #4
larowlanComment #5
larowlanAh I did make an alpha release, I didn't realise. Either way updated the project page to point to your module and added a note about the core issue.