Add editor shortcuts for Tiles and Objects navigation (SuperTux#2988)

[VaLLeX8]

• Add "Q" to switch between Tiles and Objects;
• Add Shift + "Q" to toggle between current and last subgroup inside the same group (Tiles or Objects);
• Add Shift + Scroll to open the current menu and select on release;
• Add "S" to open a menu of sub-subgroups within each Tiles subgroup;
  • This uses the divisions previously commented in tiles.strf.

Co-authored-by: Afonso Mateus [email protected]

[swagtoy]

Did you attempt to merge all of master and stage it? You'll have to fix that, as it's difficult to review the code

[VaLLeX8]

Yes, you are absolutely right. I think it's good now.

[bruhmoent]

I'm honestly not a fan of introducing more shortcuts to the editor unless a versatile solution is implemented which allows the user to bind each individual key to their wishes, and preferably most of them would be bound to none from the start. I find it confusing to have unknown bindings all over the keyboard which perform vastly different actions, and inconvenient.

[MatusGuy]

In the meantime, I would still prefer to at least have some sort of shortcut, instead of postponing it to whenever you guys implement bind-able shortcuts.

But, until then, why is adding more shortcuts an issue? Just convert them all when the time comes...

[Frostwithasideofsalt]

Can you change the keybind for "S" to "G" instead? s overlaps with ctrl+s and WASD camera controls if you have your movement keybinds set to those.

[Frostwithasideofsalt]

apon further testing, this also seems to break all custom tilesets, alongside with breaking the "others" tile category. the broken tilemaps wont show their name, instead showing a empty space where the tilemap would be.

these broken tile-groups also contain no tiles in the editor.

any existing tiles from these groups still work in the levels, they just dont show up in the editor.

[VaLLeX8]

@Frostwithasideofsalt, when adding custom tilesets, you still add them to your "tiles.strf", right? The problem I see there is that the menu now searches for "parent_group" and not "name". My guess would be that those tilesets don't have a defined parent_group in that file, which causes them to appear blank in the menu and not show the tiles when you select them. If that's the case, you just need to add something like "(parent_group (_ ""))" above the "(name (_ ""))".
If that doesn't solve the problem, please let me know—and thank you for taking a look at this.

[Frostwithasideofsalt]

There really needs to be a fallback if it can't find a "parent_group" to ensure that all the old custom tilesets dont break.

[VaLLeX8]

Ok. I'll do that.

[Frostwithasideofsalt]

alright, thank you!