[Bug]: miniApp.mountSync() also mounts themeParams

[erfanmola]

Telegram Application

Telegram for Android

Describe the Bug

Calling miniApp.mountSync() also does mount the themeParams, making themeParams.isMounted() to be true, so codes like this will fail:

To Reproduce

		if (miniApp.mountSync.isAvailable() && !miniApp.isMounted()) {
			console.log("Bind Mini App");
			miniApp.mountSync();
			bindMiniAppCssVars();
		}

		if (themeParams.mountSync.isAvailable() && !themeParams.isMounted()) {
			console.log("Bind Theme");
			themeParams.mountSync();
			bindThemeParamsCssVars();
		}

This is not the case for viewport.mount(), these should be documented somewhere to not find it by accident or bug.

Expected Behavior

if themeParams is being mounted as a part of miniApp, then why separate them? if they need to be separate, their functinalities and mounting should be separate too.

[heyqbnk]

What solution do you suggest for this problem?

[erfanmola]

What solution do you suggest for this problem?

Why would miniApp.mountSync() mount the themeParams in first place?

[heyqbnk]

The reason is the miniApp component depends on some values from themeParams, that's why it should be mounted - to use the actual values. We could throw an error if themeParams were not mounted, but to do it, we should probably define a new mount function as long as this change will break backward compatibility.