Adding some description to the items defined in the following image will improve the usability of this module:

CommentFileSizeAuthor
#10 interdiff.txt8.16 KBmarcoscano
#10 2779577-10.patch9.59 KBmarcoscano
#10 better_descriptions_w_defaults.png76.05 KBmarcoscano
#9 better_descriptions.png69.47 KBmarcoscano
#9 interdiff.txt3.79 KBmarcoscano
#9 2779577-9.patch3.93 KBmarcoscano
Selection_150.png51.84 KBdagmar
#4 provide-descriptions-for-EB-wizard-step1-2779577-3.patch3.25 KBmarcoscano
#6 interdiff-2779577-3-6.txt2.21 KBdagmar
#6 provide-descriptions-for-EB-wizard-step1-2779577-6.patch3.36 KBdagmar

Comments

dagmar created an issue. See original summary.

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
commented
slashrsm’s picture

slashrsm commented
Category: Bug report » Task
Issue tags: +D8Media, +Novice, +UX

True. A description under each drop-down or even a general help text.

Could rely on relevant docs in https://GitHub.com/drupal-media/d8-guide.

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented
Status: Active » Needs review
StatusFileSize
new provide-descriptions-for-EB-wizard-step1-2779577-3.patch3.25 KB

Wording is hard :) but here is a first proposal.

On the other hand, wouldn't some "common" defaults for these values also ease the use for users non-familiarized with the module? Based on the idea of "in doubt, leave the defaults". Perhaps iFrame + Tabs + No selection could be a sensible default for a very basic use.

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
commented
Status: Needs review » Needs work

Thanks @marcoscano this looks better.

+++ b/src/Form/GeneralInfoConfig.php
@@ -108,6 +119,7 @@ class GeneralInfoConfig extends FormBase {
+      '#description' => $this->t('If using more than one browser widget, the editor can select them by using horizontal tabs or a drop-down form element. Select "Single widget" if your Entity Browser will have only one widget available.'),

I think here we should be more explicit about this will be configured in the future. You are trying to make the user decide something that maybe he/she never configured in the past.

Based on the idea of "in doubt, leave the defaults". Perhaps iFrame + Tabs + No selection could be a sensible default for a very basic use.

+1 But maybe with modal instead of iFrame.

dagmar’s picture

dagmar
Primary language Spanish
Location Argentina
commented
Status: Needs work » Needs review
StatusFileSize
new provide-descriptions-for-EB-wizard-step1-2779577-6.patch3.36 KB
new interdiff-2779577-3-6.txt2.21 KB

What about this?

slashrsm’s picture

slashrsm commented
Status: Needs review » Needs work

The problem with current proposal is that it assumes only "core" plugins are available. When other modules provide more plugins they also appear in the dropdowns. It is OK to use core plugins as examples for explanation, but we shouldn't pretend that there won't be more at some point.

One idea I had was to use descriptions from plugins to display more info about each individual plugin.

slashrsm’s picture

slashrsm commented
Parent issue: » #2764875: Remove ctools dependency by building own admin UI
marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented
Status: Needs work » Needs review
StatusFileSize
new 2779577-9.patch3.93 KB
new interdiff.txt3.79 KB
new better_descriptions.png69.47 KB

True!

Attached a new proposal, not making assumptions on which plugins are installed.

To make it easier to see the what it gives graphically on the form, here you have a screenshot of the result:

Better descriptions for EB main form

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented
StatusFileSize
new better_descriptions_w_defaults.png76.05 KB
new 2779577-10.patch9.59 KB
new interdiff.txt8.16 KB

The same idea, but
- slightly rewording the descriptions of the plugins, trying to clarify a little bit more what each one does
- adding some non-empty defaults, as mentioned before

The result:

Snapshot of new descriptions, with default values

marcoscano’s picture

marcoscano
Primary language Portuguese, Brazil
Location Barcelona, Spain
commented
Issue tags: +Usability
CTaPByK’s picture

CTaPByK commented
Status: Needs review » Reviewed & tested by the community

Looks good to me. Thanks!

rajab natshah’s picture

rajab natshah
he/him
commented

+1 This will be much better.

slashrsm’s picture

slashrsm commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

  • slashrsm committed c6a53f9 on 8.x-1.x authored by marcoscano 
    Issue #2779577 by marcoscano, dagmar, CTaPByK, RajabNatshah: Add more...

Status: Fixed » Closed (fixed)

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