From dd1ffb03905b5483d15d8ea96df7694ba798ec0f Mon Sep 17 00:00:00 2001 From: Paul Licameli Date: Sat, 10 Mar 2018 18:55:02 -0500 Subject: [PATCH] Indicative mood for all verbs in command and effect descriptions... ... Some of them have no verb. --- include/audacity/IdentInterface.h | 1 + src/commands/OpenSaveCommands.h | 4 ++-- src/effects/ChangePitch.cpp | 2 +- src/effects/ChangeSpeed.cpp | 2 +- src/effects/ChangeTempo.cpp | 2 +- src/effects/Paulstretch.cpp | 2 +- 6 files changed, 7 insertions(+), 6 deletions(-) diff --git a/include/audacity/IdentInterface.h b/include/audacity/IdentInterface.h index 55f7c594c..84085aeb1 100644 --- a/include/audacity/IdentInterface.h +++ b/include/audacity/IdentInterface.h @@ -69,6 +69,7 @@ public: virtual wxString GetVersion() = 0; // This returns a translated string + // Any verb should be present tense indicative, not imperative virtual wxString GetDescription() = 0; // non-virtual convenience function diff --git a/src/commands/OpenSaveCommands.h b/src/commands/OpenSaveCommands.h index bbb54ca5a..0cd65d1ff 100644 --- a/src/commands/OpenSaveCommands.h +++ b/src/commands/OpenSaveCommands.h @@ -28,7 +28,7 @@ class OpenProjectCommand : public AudacityCommand public: // CommandDefinitionInterface overrides wxString GetSymbol() override {return OPEN_PROJECT_PLUGIN_SYMBOL;}; - wxString GetDescription() override {return _("Open a project.");}; + wxString GetDescription() override {return _("Opens a project.");}; bool DefineParams( ShuttleParams & S ) override; void PopulateOrExchange(ShuttleGui & S) override; bool Apply(const CommandContext & context) override; @@ -61,4 +61,4 @@ public: bool mbCompress; bool bHasAddToHistory; bool bHasCompress; -}; \ No newline at end of file +}; diff --git a/src/effects/ChangePitch.cpp b/src/effects/ChangePitch.cpp index 94251bce5..0a65832a5 100644 --- a/src/effects/ChangePitch.cpp +++ b/src/effects/ChangePitch.cpp @@ -122,7 +122,7 @@ wxString EffectChangePitch::GetSymbol() wxString EffectChangePitch::GetDescription() { - return _("Change the pitch of a track without changing its tempo"); + return _("Changes the pitch of a track without changing its tempo"); } wxString EffectChangePitch::ManualPage() diff --git a/src/effects/ChangeSpeed.cpp b/src/effects/ChangeSpeed.cpp index d9e44645b..ffe9005f1 100644 --- a/src/effects/ChangeSpeed.cpp +++ b/src/effects/ChangeSpeed.cpp @@ -112,7 +112,7 @@ wxString EffectChangeSpeed::GetSymbol() wxString EffectChangeSpeed::GetDescription() { - return _("Change the speed of a track, also changing its pitch"); + return _("Changes the speed of a track, also changing its pitch"); } wxString EffectChangeSpeed::ManualPage() diff --git a/src/effects/ChangeTempo.cpp b/src/effects/ChangeTempo.cpp index 9f8959191..26b4d11f2 100644 --- a/src/effects/ChangeTempo.cpp +++ b/src/effects/ChangeTempo.cpp @@ -99,7 +99,7 @@ wxString EffectChangeTempo::GetSymbol() wxString EffectChangeTempo::GetDescription() { - return _("Change the tempo of a selection without changing its pitch"); + return _("Changes the tempo of a selection without changing its pitch"); } wxString EffectChangeTempo::ManualPage() diff --git a/src/effects/Paulstretch.cpp b/src/effects/Paulstretch.cpp index 14e075680..9f8cc3a3d 100644 --- a/src/effects/Paulstretch.cpp +++ b/src/effects/Paulstretch.cpp @@ -105,7 +105,7 @@ wxString EffectPaulstretch::GetSymbol() wxString EffectPaulstretch::GetDescription() { - return _("Use Paulstretch only for an extreme time-stretch or \"stasis\" effect"); + return _("Paulstretch is only for an extreme time-stretch or \"stasis\" effect"); } wxString EffectPaulstretch::ManualPage()