Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ifrik’s picture

See #2030569: [policy] Decide how to refer to "entities" and "bundles" in D8 UI
and the help text for Link module #2028799: Improve help for link module, for the default wording of modules that provide fields for fieldable entities.

About
The Foo module allows you to create fields that contain [something]. See the <a href="!field">Field module help</a> and the <a href="!field_ui">Field UI help</a> pages for general information on fields and how to create and manage them. 
jover’s picture

Status: Active » Needs review
FileSize
1.34 KB

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

petrpo’s picture

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

berkas1’s picture

Status: Needs review » Needs work

Changing to Needs work per #3

batigolix’s picture

Component: documentation » number.module
Issue summary: View changes
Status: Needs work » Needs review
Parent issue: » #1908570: [meta] Update or create hook_help() texts for D8 core modules
Related issues: +#2028799: Improve help for link module
FileSize
2.4 KB
2.44 KB

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

batigolix’s picture

I found there was a period missing.

Interdiff interdiffs between #2 and #6

batigolix’s picture

Removed some references to module that I copy-pasted from

jhodgdon’s picture

Status: Needs review » Needs work

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

batigolix’s picture

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.

batigolix’s picture

Status: Needs work » Needs review
jhodgdon’s picture

Status: Needs review » Needs work

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.

batigolix’s picture

Status: Needs work » Needs review
FileSize
2.73 KB
2.46 KB

Patch addresses points in #11

jhodgdon’s picture

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...

jhodgdon’s picture

Status: Needs review » Needs work
batigolix’s picture

Status: Needs work » Needs review
FileSize
2.46 KB

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.

jhodgdon’s picture

Issue tags: +Needs manual testing, +Novice

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

jhodgdon’s picture

Status: Needs review » Needs work

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! :)

batigolix’s picture

batigolix’s picture

Status: Needs work » Needs review
jhodgdon’s picture

Component: number.module » documentation
Status: Needs review » Reviewed & tested by the community

Thanks! This one is ready to go in.

jhodgdon’s picture

Status: Reviewed & tested by the community » Fixed

Thanks again everyone! Committed to 8.x.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.