Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a keybinds for topskymaps #468

Closed
janfelhut04 opened this issue Oct 8, 2024 · 7 comments
Closed

Add a keybinds for topskymaps #468

janfelhut04 opened this issue Oct 8, 2024 · 7 comments
Assignees

Comments

@janfelhut04
Copy link

I've seen it is possible to add keybinds for topsky maps. Is it possible to add keybinds for MVA maybe others as well.

Here would be a code example to toggle ENGM MVA with ALT + M.

Shortcut_RadarMenu_Combo=0x00
Shortcut_Map_Xfr_Key=1,0x4D ; M
Shortcut_Map_Target=1,MVA\MVA ENGM

This could be very helpfull if we only need the MVA at a specific moment.
I would suggest creating a new topsky_map with all MVAs in Norwy and use this when toggling

@github-project-automation github-project-automation bot moved this to Unassigned in ENOR Sectorfile Oct 8, 2024
@janfelhut04 janfelhut04 changed the title Add a key binds for topsky Add a keybinds for topskymaps Oct 8, 2024
@janfelhut04
Copy link
Author

The codes are placed in TopskySettings and get all the data from TopskyMaps

@morettio
Copy link
Collaborator

morettio commented Oct 9, 2024

I've tested it and was thinking of doing something like that on other maps. I don't see why we need it on MVA maps since the zoom setting is already implemented. Many think that opening the radar menu and going to maps is too many clicks, and few remember all the shortcuts if we add more of them.

@janfelhut04
Copy link
Author

Is it at least possible to remove MVA apparance for TWRs? I don't see the reason why we should show MVA for TWR positions. This could be add manually in my opinion. Showing the VFR reporting points as TWR is more than enough.
Skjermbilde 2024-10-11 161731

@morettio
Copy link
Collaborator

I have tested it, and the best solution is adding some lines to the TopSkySettingLocal.txt file.

If we are going to hide it to all _TWR we also need to add it to all XXXX_Y_TWR/GND/DEL and so on. So the best solution to the sectorfile is as it is now.

image

@morettio
Copy link
Collaborator

morettio commented Oct 11, 2024

This is because Topsky reads the setting.txt file first and then the localsettings.txt file afterwards.

morettio added a commit that referenced this issue Oct 11, 2024
need more info from developer to group maps in a group. This is necessary to show/hide multiple maps on one shortcut.
@morettio morettio self-assigned this Oct 11, 2024
@morettio morettio moved this from Unassigned to In progress in ENOR Sectorfile Oct 11, 2024
@morettio
Copy link
Collaborator

morettio commented Oct 11, 2024

  • Check if the developer has a solution on the plugins forum
  • Add MVA maps on shortcut

morettio added a commit that referenced this issue Oct 30, 2024
Toggle all MVA maps with ALT + M
@morettio
Copy link
Collaborator

Added Toggle all MVA maps with ALT + M

@github-project-automation github-project-automation bot moved this from In progress to Done in ENOR Sectorfile Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants