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

Create index.d.ts #106

Closed
wants to merge 26 commits into from
Closed

Create index.d.ts #106

wants to merge 26 commits into from

Conversation

runette
Copy link

@runette runette commented Dec 21, 2019

Create an index.d.ts to help developers using Typescript.

There is a typedef at @types/leaflet-fullscreen but that is minimal. This version is complete and having it in the repo puts control back in the hands of the original devs.

@runette
Copy link
Author

runette commented Dec 31, 2019

I rolled up all of the PRs into this one repo - since there does not seem to be very much maintenance activity on this repo :

runette and others added 20 commits January 2, 2020 12:43
Replaced depricated event 'fullscreenchange'
Based on documentation, the added method to Map is called `isFullscreen` and not `isfullscreen`.
[accessibility] Align with Leaflet controls
Existing implementation handle icon with PNG, with manual positioning
with CSS background image.

This PR rework icons with Font Awesome SVG:

  - Locally venor Font Awesome v5.15.4 `expand-solid.svg` and
    `compress-solid.svg`
  - Rework styling with SCSS
  - Fix accessibility (see Leaflet#110)
  - Test with Leaflet v1.7.1
  - Update demo with above changes

Fix Leaflet#98
Fix Leaflet#102
Fix Leaflet#103
Fix Leaflet#110
Fix Leaflet#114

Signed-off-by: Wong Hoi Sing Edison <[email protected]>
@runette runette closed this May 14, 2022
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 this pull request may close these issues.

5 participants