Options.module's short description says only:
'Defines selection, check box and radio button widgets for text and numeric fields.'
while in D8 it actually does:
- defines Boolean and various List field types
- defines selection, check box and radio button widgets for Boolean and various List fields.
The wording is a problem because:
- people looking for these nice widgets won't find them when adding a text or numeric field, as the description suggests.
- people looking for a Boolean type will search forever until they try Options module
The help text is not just a bit more verbose, but also a bit more correct:
'The Options module defines various fields for storing a list of items, for use with the Field module. Usually these items are entered through a select list, checkboxes, or radio buttons. See the Field module help page for more information about fields.'
Only the Boolean fields are missing there.
In D7, where list field types were provided by a separate List module, this was even worse. Enabling Options.module you would search forever and won't find the nice widgets anywhere in the UI. Therefore certainly needs backport.
Comment | File | Size | Author |
---|---|---|---|
#1 | D8_options_module_description-2020691-1.patch | 1.63 KB | Pancho |
#1 | D7_options_module_description-2020691-1.do-not-test.patch | 3.19 KB | Pancho |
Comments
Comment #1
PanchoHere's two patches for D8 and D7.
In D7 list.info I removed the reference to options.module, because it's a dependency anyway.
The other way around, options.info needs to refer to list.module because it otherwise doesn't do anything.
All descriptions can certainly be further improved, but this should be a solid starting point.
Comment #3
Pancho758 fail(s), and 307 exception(s)? Must be kidding...
#1: D8_options_module_description-2020691-1.patch queued for re-testing.
Comment #5
Pancho814 fail(s), and 504 exception(s)? No way.
#1: D8_options_module_description-2020691-1.patch queued for re-testing.
Comment #6
Pancho#1: D8_options_module_description-2020691-1.patch queued for re-testing.
Comment #7
jhedstromI'm unsure of exposing the term boolean to the UI. Regardless, this needs a reroll.
Comment #16
quietone CreditAttribution: quietone as a volunteer commentedThis work has moved into a later issue, #3060616: Update the module descriptions on the Extend page to fit the Help texts. Closing as a duplicate.