I have no idea what this functionality does, what I do know is that the text can be better, I have rewritten it so it only uses the label.

before-views-embed-display.png

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bojhan’s picture

Status: Active » Needs review
FileSize
697 bytes

Status: Needs review » Needs work
Issue tags: -Usability, -VDC

The last submitted patch, drupal.core_.settingsscreentext.1837998.patch, failed testing.

Bojhan’s picture

Status: Needs work » Needs review
Issue tags: +Usability, +VDC
dawehner’s picture

Well ... in fact you can use the embed display without this checkbox (it's just not visible in the UI), but yeah, for the normal user
don't have to know / don't want to know.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

So i think the new text is okay even it's not really the truth.

xjm’s picture

Status: Reviewed & tested by the community » Needs review

I don't understand the new text either. Also, "allow using" is awkward.

dawehner’s picture

What about keeping the #title from now and skip the description?

xjm’s picture

I think what we need is something like:

Allow embed displays to be added to Views through the user interface.

I am trying to understand why this is a setting at all, and what it's for. Drupal\views\Plugin\views\display\Embed is, like, empty.

xjm’s picture

Essentially, there is nothing that tells the user what an "embed display" is. Or me.

Bojhan’s picture

I don't get this setting either, honestly - I just figured it sounded weird.

dawehner’s picture

I am trying to understand why this is a setting at all, and what it's for. Drupal\views\Plugin\views\display\Embed is, like, empty.

There are reasons we try to hide this from the user :)

It's doing nothing specia! You have a separated display which you can embed using php. All other display plugin types does someone additional, which you don't want in pure php code.

xjm’s picture

Maybe:

[ ] Configure embeddable displays in the user interface.
Embeddable displays are code-only displays that do not display on your site by default, but can be used by another module.

That's an awful lot of text, but we need to explain what it is somehow. A big problem here is that the views term "display" is exactly what it isn't semantically in this case. ;)

xjm’s picture

FileSize
13.69 KB
1.55 KB

Let's try this. I removed some of the other mostly-redundant descriptions so there's less text to read in this section, but included a description for this option.
embed_display.png

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

You seem to know that you shouldn't put "." at the end of the sentence. At least for me this new texts would help.

tim.plunkett’s picture

Title: Show the embed display in the ui. » Show the embed display in the UI

Speaking of which.

Dries’s picture

Let's see what Bojhan thinks.

catch’s picture

Assigned: Unassigned » Bojhan
Bojhan’s picture

Status: Reviewed & tested by the community » Needs work

We cleaned it up a lot, thanks xjm! However this could probably be a bit better, since its not common to have a checkbox that says "Configure X in the UI" and it could probably be shorter. What about:

"Allow embedded displays"
Embedded displays are code-only displays, often used by modules to do X.

I guess, you don't actually allow it? But we should probally say something along the lines of a "Show" or "Allow" - having it be "Configure X in the UI" is a bit confusing, because its so uncommon.

Bojhan’s picture

Assigned: Bojhan » Unassigned
dawehner’s picture

Status: Needs work » Needs review
FileSize
812 bytes
1.46 KB

What about showing the user how to actually use embed displays?

xjm’s picture

Hmm, I'm not sure that's a good idea. The audience of UI helper text is not developers.

tim.plunkett’s picture

Well, the point is that the embed display is only useful when using views_embed_view() anyway...
I like it.

damiankloip’s picture

Status: Needs review » Needs work

I think the helper text can be for developers too, why not?! Developers do use Drupal too, sometimes... :) What if you have more of a 'dabbling' developer... this would be a good guide I think.

I'm also not sure why we are removing all the description text, this makes the options sound more vague than before. Which I would say is less helpful. For example, "Always show the master (default) display" is not that indicative of what it does. We lose the where and the why.

Bojhan’s picture

@damiankloip Agreed, lets work on it some more. Its natural though, sometimes we cut too much and often not enough :)

damiankloip’s picture

Opened #1882114: Remove listing filters option from views settings earlier, this removes another option, which I guess helps :)

dawehner’s picture

Issue tags: +Novice, +Needs reroll
David Hernández’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
1.66 KB

Re-rolled

dawehner’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Usability, -VDC

Nice!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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