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

new class L.PortalMarker to incapsulate all portal visual interactions #251

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

johnd0e
Copy link
Contributor

@johnd0e johnd0e commented Sep 10, 2019

  • class L.PortalMarker (todo: document )
    methods:
    • setStyle (deprecated, for compatibility with old highlighters)
    • setMarkerStyle
    • setSelected
  • deprecated:
    • window.createMarker
    • window.setMarkerStyle
  • related functions (next candidates for refactoring/deprecation)
    • window.setPortalIndicators
    • window.selectPortal
    • window.portalMarkerScale
    • window.getMarkerStyleOptions

johndoe added 4 commits February 21, 2019 10:10
`setStyle` is redefined in order to suppress re-render in `highlightPortal`
if selected - 3rd re-render avoided too
* `setMarkerStyle` is now method of L.PortalMarker
* minor rearrangements
@johnd0e johnd0e added the development general development issue label Sep 10, 2019
@johnd0e johnd0e added the WIP Work in progress || Proof of the concept label Sep 10, 2019
@modos189 modos189 force-pushed the master branch 3 times, most recently from a34b800 to 7b9edd5 Compare December 12, 2020 07:21
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request Feb 15, 2021
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request Feb 17, 2021
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request Feb 17, 2021
apply/adapt set(Marker)Style PR IITC-CE#251 by johnd0e

commit 6201837
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request Feb 17, 2021
apply/adapt set(Marker)Style PR IITC-CE#251 by johnd0e

commit 6201837
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request Feb 17, 2021
apply/adapt set(Marker)Style PR IITC-CE#251 by johnd0e

commit 6201837
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request May 6, 2021
apply/adapt set(Marker)Style PR IITC-CE#251 by johnd0e

commit 6201837
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request Sep 30, 2021
apply/adapt set(Marker)Style PR IITC-CE#251 by johnd0e

commit 6201837
le-jeu added a commit to le-jeu/ingress-intel-total-conversion that referenced this pull request Jan 8, 2022
apply/adapt set(Marker)Style PR IITC-CE#251 by johnd0e

commit 6201837
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development general development issue WIP Work in progress || Proof of the concept
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant