Implement open save menu and save menu access management commands - #815
Conversation
AerunDev
left a comment
There was a problem hiding this comment.
Hey ! J'ai pu tester la PR, et pour moi tout est bon, bien joué !
Une remarque tout de même au niveau du wording entre "Instructions" affiché dans le titre de la bibliothèque de commandes, et "commandes" dans le message dans le cas où un éditeur n'a pas de champ à afficher.
Il faudra qu'on voit si on souhaite harmoniser ou non.
J'ai fait un commit pour harmoniser les termes anglais avec le français, du coup j'en ai profité pour changer "commande" par "instruction" partout, ce sera plus cohérent si on utilise les mêmes termes partout dans l'appli. |
Excellente remarque. Pour clarifier : que le terme reste "Command" dans le code me pose aucun problème. En revanche, il faudra prendre bien soin de n'utiliser et n'afficher que le terme "Instruction(s)" dans les interfaces. C'est important pour éviter les homonymes et la confusion. On est dans un univers technique, "Commande" est vaste et fait déjà référence à d'autres objets, les commandes de script ou des commandes "/" dans un terminal par exemple. On construit un flux algorithmique, ce qui est parfaitement une suite d'instructions. J'essaie moi-même de m'efforcer à utiliser ce terme dans les maquettes et la documentation. Essayons d'harmoniser ça ! |

Description
This PR adds two commands to the list of available ones, Open Save Menu and Manage Save Menu Access.
closes #580
closes #581
closes #597
Tests to perform