Skip to content

Added label, title, info box to markers#93

Closed
brentbiseda wants to merge 4 commits intogmplot:masterfrom
brentbiseda:master
Closed

Added label, title, info box to markers#93
brentbiseda wants to merge 4 commits intogmplot:masterfrom
brentbiseda:master

Conversation

@brentbiseda
Copy link
Copy Markdown

No description provided.

@ralienpp
Copy link
Copy Markdown

This is a nice feature, I've installed this version separately because I needed it in one of my experiments. What are the impediments for merging this pull request?

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 like this PR has many lines that shouldn't be removed or are pulled in from other PRs (e.g. function get_cycle()); caused by a rebase onto a different branch?

@brentbiseda, marker titles should already be implemented; if you could figure out what happened with the missing/extra lines here, I can review and merge the label and info box changes.

@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 set a separate title, label, and InfoWindow content for each marker, instead of always showing the same content for both title and label and showing the label content in the InfoWindow.

@frslm frslm closed this Jun 7, 2020
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.

4 participants