Update hook_help for Text module

Trying to correct it according to standards

Looks OK

Yes, this still needs work. Check out the help for the Link or Telephone modules for examples.

Yes, it is impossible to *disable* a module, but you can still *uninstall* a module. But I think it's probably OK to leave that reference as it is.

Anyway, this mostly looks good! A few typographical/grammatical fixes would help:

a)

or allow users to formated text.

formated -> format

b)

Text formats and editors page </a>. 

Extra space in the A link text.

c) "formated" is misspelled. Should be "formatted".

d) Heading "Setting and display the text length"... This doesn't make sense to me. Reading what is in that item, maybe it should be just "Setting the text length"?

e) You can change the number of characters that define the maximum text length
This could be simplified to say "You can set the maximum text length"

f) can be entered on the <em>Manage Extra space before the EM tag.

g) I do not think it is clear in the last Uses point what the difference is between the maximum text length and the size of the form field.

h) choose whether to displayed a trimmed version of the text, and if so after how many characters the text should be cut off.
- displayed -> display
- And that last phrase... would be better as "and if so, where to cut off the text"

The patch file is empty. Can you re-upload?

So... In reading this one more time, I'm a bit concerned about the first line of About. I think we should list the actual field types rather than saying "various" maybe? List the names someone would see in the Field UI and maybe add a brief explanation of what each one is? I don't know... it is possibly OK as it is.

Other than that... looks pretty good! I noticed a typo:
"You can set change the maximum text length"

And the Text with Summary field has some additional options that should maybe be described?

This help text overall seems very good to me! There are a few minor grammatical things that need fixing in this patch:

a) This sentence no verb (start of About):
"The Text module allows you to short and long text fields with optional summaries."

b) In Uses item 1:
"If you choose Text as field type on the Manage fields page,"
needs "the" before "field type"

c) Uses item 2:
" you can set the number of rows that is displayed to users."
is -> are

d) Uses item 4:
"choose whether to display a trimmed version of the text, and if so, where to cut off the text."
Normally I think "whether" should have an alternative, so it should be "whether ... or ...", but there is no "or" alternative given. Maybe just leave out the word "whether"?

e) Uses item 5:
"In order to display a summary that is different from the trimmed beginning of text"
This seems very awkward to me. Maybe:
"As an alternative to using a trimmed version of the text, you can choose to enter a separate summary by ..."

f) Uses item 5:
"by choosing the appropriate format on the Manage Display page."
There is an extra space between "the" and "appropriate"

g) Last Uses item:
"by choosing Plain text as Format on the Manage Display page."
Needs "the" before "Format".

Excellent!

One thing:

As an alternative to using a trimmed version of the text, you can enter a separate summary by choosing the Long text with summary format on the Manage Fields page.

I don't think this is a "format" -- isn't it a "field type"?

Other than this small thing, I think this is ready to go (it will still need a manual test however). Thanks!

Excellent, thanks! I think this is ready for someone to give it a manual test:
- Verify that the formatting is good.
- Verify that all the links work.
- Verify that where specific page names, tab names, labels, etc. are mentioned in the help text, they match what a user actually sees on the screen.

- formatting is good
- all the links work.
- specific page names, tab names, labels, etc. are correct but I had to change the case to make them match correctly

20: text-updated_help_d8-documentation.2091461-20.patch queued for re-testing.

Looks good, thanks! Assuming the tests pass this time (the failure seemed to be totally unrelated), I think this is ready to go in.

Thanks again everyone! Committed to 8.x.