1. The module description is very long, very technical and pretty scary: Provide a framework for various different types of embeds in WYSIWYG editors, common functionality, interfaces, and standards.
    But we don't say WYSIWYG editors in Drupal 8, we say text editors. This module should be consistent with that, so that it's easier for end users to understand the relation.
  2. The link at /admin/config looks like this:

    It's great that it's listed under content authoring. But just embed buttons is rather meaningless. Embed what? Where? These buttons are specifically for text editors. So let's say Text editor embed buttons. And let's put it below Text formats & editors, so that there is a logical flow to reading this, to discovering this.
CommentFileSizeAuthor
#2 2624450-2.patch955 byteswim leers
#2 Screen Shot 2015-11-27 at 16.47.30.png24.42 KBwim leers
Screen Shot 2015-11-27 at 16.45.53.png22.47 KBwim leers

Comments

Wim Leers created an issue. See original summary.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Active » Needs review
StatusFileSize
new Screen Shot 2015-11-27 at 16.47.30.png24.42 KB
new 2624450-2.patch955 bytes

Addressed both. Screenshot for point 2:

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Issue summary: View changes
slashrsm’s picture

slashrsm commented
Status: Needs review » Fixed

Committed. Thanks!

Status: Fixed » Closed (fixed)

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