jaykaywp

Brilliant, thanks for the swift reply

Ok, thanks for looking into this

Thanks for getting back to me, I’ll speak to my host and see if we can get to the bottom of it.

Yes I can see how maintaining cross-browser compatibility is a pain! Thanks again for all your efforts.

Thanks for the comprehensive reply! Much appreciated

Hello there, just to say there is a small bug with this update on Mac Safari when you hover the cursor over the profile, it doesn’t follow smoothly often just stopping. Apologies I didn’t test this on previous update so not sure if it was also on that one, it works fine on 1.0.11

Thanks

Hello, theses are the errors I get on iOS…

[Error] TypeError: undefined is not an object (evaluating 'screenfull.raw.fullscreenchange')
	_createButton (Control.FullScreen.min.js:1:1934)
	onAdd (Control.FullScreen.min.js:1:653)
	addTo (leaflet.js:5:46460)
	addControl (leaflet.js:5:46969)
	(anonymous function) (Control.FullScreen.min.js:1:3507)
	(anonymous function) (leaflet.js:5:3515)
	initialize (leaflet.js:5:26308)
	i (leaflet.js:5:2622)
	(anonymous function) (leaflet.js:5:141670)
	init_datahub_map0 (wordpress-plugin-for-ordnance-survey-maps:64)
	Global Code (wordpress-plugin-for-ordnance-survey-maps:108:105)
[Error] TypeError: null is not an object (evaluating 'elem.appendChild')
	init (os-api-branding.js:50)
	(anonymous function) (os-api-branding.js:61)

Could you perhaps try something like BrowserStack to test? Should be free for an Open Source project like this – https://www.browserstack.com/open-source?ref=pricing

Thanks

Hello, I can confirm that the latest version doesn’t work at all for me on iPhone. I’ve tested the maps on the plugin homepage and I just get a grey box with the control buttons or just a white space where the map should be.

On Mac Safari, it’s the same for me as @robinb415 highlighted.

For me personally, if it’s a choice between one or the other, I would prefer to have the previous implementation where it worked fully on iPhone but the full screen was a bit glitchy on Mac Safari, rather than it not working at all on iPhone.

Also just to confirm the demo – https://brunob.github.io/leaflet.fullscreen/ – works fine for me on iPhone and Mac (the fullscreen button doesn’t remove the toolbars on iPhone Safari but it works fine).

Hope that helps!

That’s great, thanks!

In case it helps…the demo on https://brunob.github.io/leaflet.fullscreen/ works for me on Safari

I went for something a bit custom in the end using a walker icon so probably not for everyone! Thanks for looking into making the changes

Great, look forward to it.

Thanks

Just following up on the marker – I appreciate this may not be for everyone but in case it helps – you can actually just change it with custom css and upload your custom svg icon to your media directory or wherever you want to host it and change the url in the css accordingly…

.elevation-position-icon:before {
  background: url(where-the-svg-is-uploaded/my-custom-svg-icon.svg) no-repeat center center / contain}

Hope that helps.

Thanks

• This reply was modified 3 years, 11 months ago by jaykaywp.
• This reply was modified 3 years, 11 months ago by jaykaywp.

Great, thanks for the quick response.

I’m just investigating options at this stage, but thanks the offer of assistance, if I do go ahead with Kadence Blocks I’ll send a link to help with the custom css needed.

Thanks again

Completely understand – thanks for the reply