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

Remove invisible marker that's blocking hover styles #25

Merged
merged 2 commits into from
Feb 6, 2019

Conversation

clpalmer
Copy link

@clpalmer clpalmer commented Feb 4, 2019

As per the closed issue #24 here's a PR describing exactly the change I need. As you can see from the screenshot, the AGM marker that's created causes a 40px transparent image to be inserted in the DOM at a higher z-index than my custom overlay so my custom overlay can no longer receive hover events. This PR just sets the visibility of that dummy AGM marker to false so it won't render the unnecessary transparent image.

image

@AckerApple
Copy link
Owner

Very interesting and I now mostly understand.

The knowledge I was missing and hope I’m right on, is the agm-core or the custom overlay library used in this package, has a transparent placeholder image of sorts. And your pull request is to set that invisible marker to default invisible in essence.

This pull request and it’s exact details are very much appreciated. I can work with this.

I have promised another pull request my time this week and so too will I test and run this through.

Thanks good person, great to be finding a level we can get things done on

@AckerApple AckerApple added enhancement New feature or request good first issue Good for newcomers labels Feb 5, 2019
@clpalmer
Copy link
Author

clpalmer commented Feb 5, 2019

Yes, your understanding is correct.

There is a 40px transparent image (because of the " " set as iconUrl) that is rendered over top of the agm-overlay and is rendered in the DOM in a separate place with a separate zindex that forces it above the overlay and blocks hover.

And, as you said, this PR just disables the marker that causes it.

@AckerApple
Copy link
Owner

Ah, I appreciate you reiterating what you said especially about the iconUrl as I'd like to look into the how/why it's default set to that (I believe it was to fight extending agm-core marker directive)

I'll check it out relatively soon.

@AckerApple AckerApple merged commit 86bb840 into AckerApple:master Feb 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants