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.

Sign up for GitHub

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

Jump to bottom

<ImageOverlay> className attribute does not insert space after leaflet-generated classes #399

Closed
sairus2k opened this issue Oct 19, 2017 · 2 comments
Closed

<ImageOverlay> className attribute does not insert space after leaflet-generated classes #399

sairus2k opened this issue Oct 19, 2017 · 2 comments

Comments

@sairus2k
Copy link

Before opening an issue, make sure to read the contributing guide and understand this is a bug tracker, not a support platform.

Please make sure to check the following boxes before submitting an issue.
Issues opened without using this template will be closed unless they have a good reason not to follow this template.

  • [*] All peer dependencies are installed: React, ReactDOM and Leaflet.
  • [*] Using a supported version of React and ReactDOM (v15.x or v16.x).
  • [*] Using the supported version of Leaflet (v1.2.x) and its corresponding CSS file is loaded.
  • [*] Using the latest stable version of React-Leaflet.
  • [*] The issue has not already been reported.
  • [*] Make sure you have followed the quick start guide for Leaflet.
  • [*] Make sure you have fully read the documentation and that you understand the technical considerations.

Expected behavior

<img class="leaflet-image-layer leaflet-zoom-animated custom-class">

Actual behavior

<img class="leaflet-image-layer leaflet-zoom-animatedcustom-class">

Steps to reproduce

Insert ImageOverlay with className attribute.
<ImageOverlay className="custom-class" />

https://jsfiddle.net/z2Lutn6y/
@PaulLeCam
Copy link
Owner

Please check the behavior using Leaflet directly to make sure it's an issue caused by this lib and not upstream.

@sairus2k
Copy link
Author

It's Leaflet itself issue. It fixed already but not released yet Leaflet/Leaflet#5830

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@PaulLeCam @sairus2k