Update the UI texts for the Configuration Manager module

A first proposal, based on hook_help texts.

The patch works ! updates the UI text as intended.

@phantomvish : Thanks for your review !
But I guess we need more reviews for RTBC.

1. +++ b/core/modules/config/config.module
   @@ -33,27 +33,27 @@ function config_help($route_name, RouteMatchInterface $route_match) {
   +      $output .= '<p>' . t('You can review differences between the active configuration and an imported configuration archive on this page to ensure that the changes are as expected, before finalizing the import. This page also shows configuration items that would be added or removed.') . '</p>';
   

   What is the value of the last sentence here? Maybe:

   "Review differences between the active configuration and an imported configuration archive to ensure that the changes are as expected, before finalizing the import."

2. +++ b/core/modules/config/config.module
   @@ -33,27 +33,27 @@ function config_help($route_name, RouteMatchInterface $route_match) {
   +      $output .= '<p>' . t('You can create and download an archive consisting of all your site\'s configuration exported as *.yml files.') . '</p>';
   

   Start directly with "Create and download…"

3. +++ b/core/modules/config/config.module
   @@ -33,27 +33,27 @@ function config_help($route_name, RouteMatchInterface $route_match) {
   +      $output .= '<p>' . t('You can upload a full site configuration from an archive file. When importing data from a different environment, the site and import files must have matching configuration values for UUID in the system.site configuration item. That means that your other environments should initially be set up as clones of the target site. Migrations are not supported.') . '</p>';
   

   Don't think we should explain what you can not do here. Is there a handbook to link to maybe? Or only say: "You can only import if the UUID in the system.site configuration item is the same."

4. +++ b/core/modules/config/config.module
   @@ -33,27 +33,27 @@ function config_help($route_name, RouteMatchInterface $route_match) {
        case 'config.import_single':
   ...
   +      $output .= '<p>' . t('You can import a single configuration item by pasting it in YAML format.') . '</p>';
   

   Remove the "You can". Try to start with a verb to get to the point asap :)

Thanks @yoroy, I agree and applied your comments.

I can see "You can" word as a prefix on few tabs on the page.Needs work(attached Screenshot).

Linking #2247291: Reorder tabs in configuration UI as if that rearranges and renames some of the tabs - this may have a knock on impact on this issue.

Thanks @manauwarsheikh, #10 applied

Thanks for the quick updates, I'm afraid I have some more changes to propose :-) Together with Eli-T we put #2247291: Reorder tabs in configuration UI back on track to get it rtbc and while reviewing we also had another good look at all the help texts. Here's the latest proposal with one additional change:

Synchronize page:
Compare the configuration uploaded to your staging directory with the active configuration before completing the import.

Full import:
Upload a full site configuration archive to the staging directory. It can then be compared and imported on the Synchronize page.

Single import:
Import a single configuration item by pasting its YAML structure into the text field.

Full export:
Export and download the full configuration of this site as a gzipped tar file.

Single export:
Choose a configuration item to display its YAML structure.

And:
Rename the "old/new" table headers to "active/staged" in the view differences modal.

#14 applied.

active/staged

is much better indeed

Thanks @pguillard, I'll take a look.

+++ b/core/modules/config/src/Controller/ConfigController.php
@@ -143,8 +143,8 @@ public function diff($source_name, $target_name = NULL, $collection = NULL) {
+        array('data' => t('staged'), 'colspan' => '2'),

Any reason the case has changed here? Whilst this will be displayed uppercase either way due to text-transform: uppercase; in the theme, if someone is using a different theme, these headings will be displayed completely lowercase.

The rest of the patch looks good and makes the changes as suggested in #14.

However, we also need to remove the

Use the export button below to download your site configuration.

text on the Export Full archive tab as it's completely redundant.

I can help here

text on the Export Full archive tab as it's completely redundant.

done

Any reason the case has changed here?

done

Thanks @a.milkovsky!

Can you add an interdiff please? https://www.drupal.org/documentation/git/interdiff

Adding interdiff

Patch in #19 implements all of the suggestions made by @yoroy, looks good to me.

Thanks all. Yes, this is good to go.

Just come across #2487588: Move CMI import/export directory "staging" to "sync", as it is confused with staging environments which proposes different changes to some of these same strings.

Postponed #2487588: Move CMI import/export directory "staging" to "sync", as it is confused with staging environments so this one can get in first.

Reviewed the changes as well, and I like!

Committed f8e3bf2 and pushed to 8.0.x. Thanks!

Nice!