[Meta] Update the module descriptions on the Configuration page

Is this task still being worked on?

I will pick this up (DrupalCon Barcelona 2015 mentored sprint - first time sprinter!) and see if I can do some more work on the patch.

This is the contributor task I *think* I'm doing!
https://www.drupal.org/node/2276597

The attached file lists the *links.menu.yml files that need to be reviewed and updated where required, identified using the following command:

find . -name "*links.menu.yml" -print | xargs grep "parent: system.admin_config" | awk -F: '{print $1}' | sort -u > 2570997-files_list.txt

All files listed are currently marked as TODO - I will update as DONE the ones that I get through today (including those that have already been picked up by ifrik).

ETA: working with Andrew Gleeson who has picked up related issue 2276599 and is also a first time sprinter

This patch includes UI help text changes for files marked as STATUS = DONE and CHANGED = Y in the attached files list txt file. I have checked most, but not all, of these changes in a clean Drupal 8 install. In some cases I couldn't figure out where the description would appear in the interface.

First time sprinter, so hopefully I haven't changed anything I shouldn't have or made anything worse!

Attached a screenshot after #14 and the changes of help text to match standards do appear to have been applied.

I have also reviewed the code and the only items that are changed are the descriptions and they do all match the the help text standard.

Adding a beta evaluation.

Looking pretty good! A few suggestions:

1. +++ b/core/modules/aggregator/aggregator.links.menu.yml
   @@ -1,6 +1,6 @@
   +  description: 'Add feeds to gather external content and configure how often they are updated.'
   

   I think "add feeds" is misleading here -- it sounds like you would be *providing* a feed from your site.

   The previous description did not have this problem of being possibly misleading.

2. +++ b/core/modules/ban/ban.links.menu.yml
   @@ -1,6 +1,6 @@
   +  description: 'Ban visits to the site from individual IP addresses.'
   

   I think you can also ban ranges of IP addresses with *, can't you? maybe not... this may be overly specific?

3. +++ b/core/modules/config_translation/config_translation.links.menu.yml
   @@ -1,6 +1,6 @@
   -  description: 'Translate the configuration.'
   +  description: 'Translate configuration items.'
   

   hm, really, configuration items? How about just

   Translate site configuration

4. +++ b/core/modules/image/image.links.menu.yml
   @@ -1,5 +1,5 @@
   +  description: 'Configure styles to manipulate the display of images.'
   

   Could be shortened to

   Configure image display styles

5. +++ b/core/modules/language/language.links.menu.yml
   @@ -1,6 +1,6 @@
   +  description: 'Configure languages for content, interface and configuration.'
   

   needs comma before "and"

6. +++ b/core/modules/path/path.links.menu.yml
   @@ -1,6 +1,6 @@
   +  description: 'Add custom URLs to existing internal paths.'
   

   Will people know what "internal paths" are?

7. +++ b/core/modules/simpletest/simpletest.links.menu.yml
   @@ -1,6 +1,6 @@
   +  description: 'Run tests against the site core code and enabled modules to assure that your code works as designed.'
   

   assure is not right here.

8. +++ b/core/modules/system/system.links.menu.yml
   @@ -40,14 +40,14 @@ system.admin_config_media:
   +  description: 'Set paths to store uploaded files, including translation files, and configure how they are accessed.'
   

   paths? Really it's directories on the web server, I think?

9. +++ b/core/modules/system/system.links.menu.yml
   @@ -56,7 +56,7 @@ system.admin_config_services:
   +  description: 'Configure the description and content of feeds provided by the site.'
   

   hm. Does it really affect all feeds (created by Views for instance, which is now in core) or just some specific ones?

10. +++ b/core/modules/system/system.links.menu.yml
    @@ -67,19 +67,19 @@ system.admin_config_development:
    +  description: 'Clear caches, configure page caching for anonymous users, and enable bandwidth optimization for CSS and JS files.'
    

    JS -> JavaScript

11. +++ b/core/modules/system/system.links.menu.yml
    @@ -91,26 +91,26 @@ system.admin_config_regional:
    +  description: 'Set whether the site has a default country and timezone, and whether users can set their own timezone.'
    

    maybe remove "whether the site has a"?

12. +++ b/core/modules/system/system.links.menu.yml
    @@ -91,26 +91,26 @@ system.admin_config_regional:
    +  description: 'Configure local site search, metadata and SEO.'
    

    comma before "and"

Careful about assure/ensure:
http://www.quickanddirtytips.com/education/grammar/assure-versus-ensure-...
I think anything we do in the Drupal UI is about "ensure", not "assure", right?

Updating with spell check, suggestion from Jhodgdon, removing additional commas etc.
Please review.

I am not convinced of almost 50% of these changes. Can we discuss this over IRC? This patch is almost to big to review.

Meanwhile, setting status...

Can u suggest @Bojhan

Updating this issue then.

So... Everyone who supplied a patch on this issue, you might want to go to the child issues and add a comment so you can be credited for the child issue patches. If you only worked on part of the patch, pick the appropriate child issue.

Reviewing on the lower level issues.

Reviewing on the lower level issues.

This needs follow-up issues created for the patch above as mentioned by @jhodgdon in comment #35

Since all child issues have been closed, I'm going to close this one as well. Please feel free to re-open if required.