skirridsystems

The issues is marked as resolved. I’ve just tested the new 7.6.0 release and it’s still not working.

As a workaround I have to disable dynamic caching before saving settings, then re-enable it.

Version 1.8.0 now uses Leaflet 1.9.4 and the fullscreen plugin is now the newer Control.FullScreen plugin. Can you see if this resolves things for you?

OK, the JS error was a red herring. It’s the CSS in Themify.

@media only screen and (max-width: 800px) {
  img {
    width:100%!important;
  }
}

The map and markers disappear when the width is 800px or less.

Forcing image width to 100% of the parent item irrespective of where the image is coming from seems like questionable programming practice.

I think you’ll have to take this up with Themify

I suspect this is something to do with your theme or plugins. If you look at the developer console there’s a jQuery error in another part of your code, which will cause JavaScript to stop executing. If that happens before your map display completes then you’ll only get part of it. Map tiles are drawn at one of the later stages.

Or maybe something to do with your caching options? Most of the JS is marked as deferred, but there is an inline chunk which is not and maybe depends on being executed after the deferred part?

Disable contact emails?

Sure, but I think it should be ‘Disable emails’, not ‘admin’ explicitly, and the comment could be:

You can disable email being sent if you only want to list submissions in your dashboard.

Simon

OK, understood. I’ll disable caching for that page.

Yes, it’s that plugin. I know I have to disable caching for any page which uses VS Event List and I was hoping not to have to do that for contact forms too.

Simon

Hi Guido,

Yes! Looks good to me ??

Thanks for fixing,

Simon

Hi Guido,

Yes, it looks like other forms don’t try to make the name/id of their honeypot fields look like anything else. Ninja Forms uses id="nf-field-hp-data.id" name="nf-field-hp" .

Alternatively, setting autocomplete="off" for those input, or even for the form itself, may help.

Simon

That is an odd one. Both OS maps and Trackserver are using Leaflet maps, but they’re using different versions, which is possibly leading to conflicts. Trackserver is using the newer version, so I can try updating the OS maps plugin to see if that helps.

Gesture handling and popup height fixed in 1.7.0

At the moment there is no way of using a separate shortcode for markers. Can you tell me more about your use case? (by email)

Marker clustering is on the roadmap but with no timescale set.

Thanks for the bug report. I have a fix for that which will appear shortly in the next plugin release. I recommend that you set Gesture Handling to Mobile-Only, otherwise mobile users will have a hard time scrolling your web pages.

The popup problem is a mix of two things: the fonts, spacing, etc you can set using CSS. I’ve updated the doc page to show this:

https://skirridsystems.co.uk/os-datahub-maps/custom-css/

Making the popup a limited height needs to be done in the Javascript. There’s a Leaflet option for max-height, beyond which you get scroll bars. I will need to experiment with this a little, but at a minimum I should be able to restrict to the map height or screen height.

Regarding the custom marker, this is possible in principle, but you need to set quite a few values to do it, so I would need to work out an appropriate way of doing that. That one may take a little longer.

Hi Gerry,

JSON files must have a .json extension, otherwise the plugin doesn’t know which file type to decode. Looks like you’ve found an omission in the documentation! I’ll add that information now.

Simon