Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Map Controls Leaflet upgrade (React 18, Leaflet 1.8.0 en React Leaflet 4.0.0) #1579

Closed
hansgrimm opened this issue May 30, 2022 · 1 comment · Fixed by #1608
Closed

Map Controls Leaflet upgrade (React 18, Leaflet 1.8.0 en React Leaflet 4.0.0) #1579

hansgrimm opened this issue May 30, 2022 · 1 comment · Fixed by #1608
Assignees
Milestone

Comments

@hansgrimm
Copy link
Contributor

hansgrimm commented May 30, 2022

Aanleiding

In Leaflet release 1.8.0 zijn een aantal issues op het gebied van toegankelijkheid opgelost, zie Leaflet/Leaflet#8021. O.a. het issue rond de sluitlink van een leaflet popup heeft geen duidelijk label “×”.

Werkzaamheden

Om Leaflet 1.8.0 te kunnen gebruiken zijn een aantal upgrades nodig:

  • React 18
  • React-Leaflet 4.0.0
  • Leaflet 1.8.0
    Voordat de gebruikerstoepassingen deze upgrades kunnen uitvoeren zal er de DSO Toolkit release moeten zijn met dezelfde versies. Concurrent versions?
@tfrijsewijk
Copy link
Member

Om deze update te realiseren moest Storybook ook naar 6.5.9.

Daarnaast moest de Stencil "React" output target worden gepatched met patch-package (ionic-team/stencil-ds-output-targets#259 (comment)).

In de DSO Storybooks is nu een error over "React 18", die kan veilig worden genegeerd volgens storybookjs/storybook#17831. Er komt binnenkort een Storybook release aan met volledige React 18 support.

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 a pull request may close this issue.

5 participants