Update hook_help for Number module

Updated D8 deprecated url() to \Drupal::url().

<a href="@field-help"> should be <a href="!field-help"> according to D8 standards. Would it be possible to correct it in last patch?

Changing to Needs work per #3

This patch rewrites the hook_help to become more similar to the one for Link module.

It adds a section that explains the 3 number field types.

It drops the sentences "Number fields can be limited to a specific set of input values or to a range of values.". It is true but not really clear. It is just some of the specific field settings (comparable to field size restrictions settings in file fields) that do not really need to be mentioned, I think

Also assigning to component number.module in case we need maintainer feedback

I found there was a period missing.

Interdiff interdiffs between #2 and #6

Removed some references to module that I copy-pasted from

Um. The patch in #7 is for options.module, not number.module? I'll wait to review until the right patch is uploaded.

Oops: working on too many issues at the same time withj too many tabs open.

Please review the patch in #6.

I'll hide the wrong patch.

Thanks -- looks mostly good! A few typographical-level mistakes:

a) This help text still says "link field" in a couple of places. It should instead use the name of the field that a user would see in this module.

There appear to be three:
Number (decimal)
Number (float)
Number (integer)

Since there are three, probably this help should say "number fields" instead of "link field"?

b) There is a list of the field types and it needs comma before "or" (in the second Uses item).

c)
"The decimal number field type allow users to enter exact decimal values. "
allow -> allows

d) e.g. means "for example", and it always needs to be followed by a comma. But it's even better just to use the words "for example", as e.g. is often misunderstood.

Patch addresses points in #11

First sentence of About:

...allows you to create fields that contain various numeric field types.

Would this be better as "that contain various types of numeric data"? I'm not sure, but it just seems a bit weird to say that fields contain field types?

Other than that, I think this is ready for a manual test...

Fixed #13.

Ready for a manual test:

- Verify that all the links work
- Verify that all mentions of pages/text within the UI match what is seen in the UI
- Verify that the formatting is OK.

Tagging for manual test, an excellent Novice task! Agreed the help is ready other than that.

I gave this a manual test today and it all looks good except for one typo:

decimal, float or, integer.

should be

decimal, float, or integer.

Whoops! :)

comma fix

Thanks! This one is ready to go in.

Thanks again everyone! Committed to 8.x.