We currently have two plugins - mediaembed and media - the naming for them is confusing.

Media's labelling is understandable enough: "Plugin for inserting images from Drupal media module"
However mediaembed's isn't so clear: "Plugin for inserting Drupal embeded media". This has nothing to do with "Drupal" or the media module.

I'd suggest "Plugin for embedding HTML snippets".

See attached patch.

CommentFileSizeAuthor
mediaembed-ux.patch422 bytesAngry Dan

Comments

johnennew’s picture

johnennew commented

This is ace! It works for me.

mike.davis’s picture

mike.davis commented
Status: Needs review » Reviewed & tested by the community

This makes it much clearer what each one is for. Works for me.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Parent issue: » #2414595: Plan for CKEditor 7.x-1.17 release

+1

adarkling’s picture

adarkling commented

+1

hass’s picture

hass commented

RTBC

  • jcisio committed 1c12311 on 7.x-1.x 
    Issue #2292575 by Angry Dan: Naming of mediaembed plugin is confusing
jcisio’s picture

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

This plugin name was mostly changed in #2159403: Make CKEditor plugin system modular and clean with another consensus there. However, I'm more ok with this suggestion, so, rename it again.

  • jcisio committed 5e5e748 on 7.x-1.x 
    Issue #2292575 mistakenly added a new line

Status: Fixed » Closed (fixed)

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

Angry Dan’s picture

Angry Dan commented