Changing MapTilerNavigationControl options have no effect about maptiler-sdk-js

[jonathanlurie]

Hello @ThatIsAPseudo , Thanks for your feeback. Originaly, the idea of MaptilerNavigationControl was to provide something simple with no confusing options and that just does the minimum, and you're right, the documentation is not aligned with this and shows the options of the parent class MaptilerNavigation instead.
The best would probably be that it accepts the same options though so I'll make a ticket for this.

[ThatIsAPseudo]

After some testing it needs some more tweaking this part :

maptiler-sdk-js/src/MaptilerNavigationControl.ts

Lines 15 to 35 in 3588c8e

	// Removing the default click event
	this._compass.removeEventListener(
	"click",
	(this._compass as HTMLButtonElementPlus).clickFunction,
	);
	// Adding custom click event
	this._compass.addEventListener("click", (e) => {
	{
	const currentPitch = this._map.getPitch();
	if (currentPitch === 0) {
	this._map.easeTo({ pitch: Math.min(this._map.getMaxPitch(), 80) });
	} else {
	if (this.options.visualizePitch) {
	this._map.resetNorthPitch({}, { originalEvent: e });
	} else {
	this._map.resetNorth({}, { originalEvent: e });
	}
	}
	}
	});

with this :

if (this._compass) {
// original code
}

Then it seems to work as expected