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

Add popup option #40

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Add popup option #40

wants to merge 7 commits into from

Conversation

kenara
Copy link

@kenara kenara commented Feb 14, 2024

This PR adds a popup option for maps showing a single point. Instead of a marker icon, an open popup is rendered containing HTML or text.

map_with_popup

Did this after encountering the follow error:

Puppeteer old Headless deprecation warning:
    In the near future `headless: true` will default to the new Headless mode
    for Chrome instead of the old Headless implementation. For more
    information, please see https://developer.chrome.com/articles/new-headless/.
    Consider opting in early by passing `headless: "new"` to `puppeteer.launch()`
    If you encounter any bugs, please report them to https://github.com/puppeteer/puppeteer/issues/new/choose.

Error opening browser
{}
TypeError: Cannot read properties of null (reading 'newPage')
If a popup parameter is set and the geojson string, parsed as an object, is a single item of type Point, render an opened popup.
If a popup is rendered, don't show the close button.
Add -P or --popup option to render an opened popup instead of a marker when GeoJSON string contains a single point
Add Handlebars custom helper for use with popup option
Add popup option
Don't show the market icon if showing popup instead.
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.

None yet

1 participant