Update (menu): Change radar display method to SetMinimapType

[hardbaited]

This should fix the minimap issue on rsg. (Only tested for rsg but should work the same with all the frameworks)

[hardbaited]

Tested on 1.15.0.beta.2 and it works

[KadDarem]

Unfortunately, RSG use this native to manage the map: https://rdr3natives.com/?_0x5FB53015

The native to get the minimap type doesn't exist. So it will be hard to fix this issue

[hardbaited]

You dont need to get the minimap type? i think? Cause when i changed the DisplayRadar(not show) native to the one rsg uses SetMinimapType(0) the minimap shows up correctly after that

[KadDarem]

RSG set the minimap type to 0. But another framework can set it to 1 or 2 or 3. So I need to get the type before opening the menu to apply again the right type after his closing. It's what I currently do with IsRadarHidden().

The real issue is : why IsRadarHidden() return true when the minimap type is set to 0.

[hardbaited]

it actually makes sense that the isRadarHidden returns true when the minimap type is 0 since 0 hides the radar. As i said in the desc of the pr i tested it only on rsg.

[KadDarem]

Unfortunately, your PR only works on RSG, not on all other frameworks/scripts that hide the map with DisplayRadar().
I'm surprised to discover that RSG doesn't use this native.

I'm sorry but I can't accept your PR for this reason.

[hardbaited]

alright should i close the pr?

[KadDarem]

Yes, you can close it.

I will investigate on other native to check if I can get the minimap type and apply the right one. Like with IsBigmapActive()

[KadDarem]

I sent a PR to R* to add the GetMinimapType() native: citizenfx/fivem#3150