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

L.TileLayer.Canvas in core #8302

Open
3 tasks done
tmiaa opened this issue Jun 20, 2022 · 4 comments
Open
3 tasks done

L.TileLayer.Canvas in core #8302

tmiaa opened this issue Jun 20, 2022 · 4 comments

Comments

@tmiaa
Copy link
Contributor

tmiaa commented Jun 20, 2022

Checklist

  • I've searched through the plugins to make sure this feature isn't already available, or think it shouldn't require a plugin.
  • I've searched through the current issues to make sure this feature hasn't been requested already.
  • I agree to follow the Code of Conduct that this project adheres to.

Motivation

I want to use the Canvas tiles. preferCanvas sounds like it should use canvas where possible.

Suggested solution

It may be that preferCanvas can also implement L.TileLayer.Canvas?

Alternatives considered

I saw #4844 / #6879 but I think they are requesting only 1 <canvas> for all the tiles, which I think is more complicated implementation.

@jonkoops
Copy link
Collaborator

If anything this would be an entirely separate TileLayer which I don't think is compatible with the philosophy of Leaflet (or it's current architecture). But I'd like to see one of the core members weigh in on this.

@Malvoz Malvoz removed the needs triage Triage pending label Jul 17, 2022
@terax6669
Copy link

http://olga.indigofloat.at/bugdemo/demo-chrome-canvas.html
This introduces white lines on FF (103.0)

@Falke-Design
Copy link
Member

This introduces white lines on FF (103.0)

I can confirm

@tmiaa
Copy link
Contributor Author

tmiaa commented Sep 23, 2022

This introduces white lines on FF (103.0)

I guess it's better to only have white lines in 3% of browsers. 🙈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants