Skip to content

Added new marker capabilities.#114

Closed
jurrutiag wants to merge 1 commit intogmplot:masterfrom
jurrutiag:marker-capabilities
Closed

Added new marker capabilities.#114
jurrutiag wants to merge 1 commit intogmplot:masterfrom
jurrutiag:marker-capabilities

Conversation

@jurrutiag
Copy link
Copy Markdown

Now a marker can have a label and an infowindow that pops up whenever the marker is clicked. The infowindow data has to be passed as a dict.

…nfowindow that pops up whenever the marker is clicked. The infowindow data has to be passed as a dict.
@jurrutiag
Copy link
Copy Markdown
Author

Some of the added things were made by you but maybe aren't in the github, i used the .py that gets downloaded using pip. Also i changed a line on the init to make the coloricon compatible on windows.

gmplot/gmplot.py Outdated
Comment on lines -1 to -16
from __future__ import absolute_import

import json
import math
import os
import requests
import warnings

from collections import namedtuple

from gmplot.color_dicts import mpl_color_map, html_color_codes
from gmplot.google_maps_templates import SYMBOLS, CIRCLE


Symbol = namedtuple('Symbol', ['symbol', 'lat', 'long', 'size'])

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems very similar to the changes made in #93.

@jurrutiag if you could address the same comment I left there, I'd be more than happy to review and merge the label and info window changes here.

@frslm frslm added the duplicate Duplicate of some other item. label Jun 7, 2020
@frslm
Copy link
Copy Markdown
Contributor

frslm commented Jun 7, 2020

Will close this as a duplicate of #119, since that PR allows users to pass in any HTML content for the InfoWindow, instead of a fixed list of key-value pairs.

@frslm frslm closed this Jun 7, 2020
@jurrutiag jurrutiag deleted the marker-capabilities branch June 7, 2020 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate Duplicate of some other item.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants