You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Original version of the plugin had a button (in visual mode) that spawned a wizardy form to let people compose an embed (choosing sidebar yes/no, set size, etc.). It was TinyMCE-based but broke at some point because of upstream TinyMCE changes. It's now been removed entirely.
@reefdog so it'd be nice to take a quick poke through and check if we think if it's architecturally sane (or get some advice from others), but yeah, i'd say put that wood behind the same arrow.
fwiw, there's some talk of shortcake becoming part of wp core but it's hard to say if/when that would happen. i'm catching up with daniel this week and can ask him if you want. otherwise i'd be a little leery re: making it a "requirement", we've looked at it but are kind of taking a wait and see approach for the moment.
@aschweigert: just as a point of note, there was a security vulnerability in the TinyMCE integration that the plugin had been using, and our solution was to just remove the wizard for the time being (it wasn't working anyway).
Would be interested if Shortcake would be worth rolling out even as a tentative feature since we currently offer no alternative.
Activity
reefdog commentedon Nov 2, 2015
Consider using Shortcake rather than rolling our own. We'd have to say "go install Shortcake if you want a wizard", but that's not a blocker.
knowtheory commentedon Nov 2, 2015
@reefdog so it'd be nice to take a quick poke through and check if we think if it's architecturally sane (or get some advice from others), but yeah, i'd say put that wood behind the same arrow.
aschweigert commentedon Nov 2, 2015
fwiw, there's some talk of shortcake becoming part of wp core but it's hard to say if/when that would happen. i'm catching up with daniel this week and can ask him if you want. otherwise i'd be a little leery re: making it a "requirement", we've looked at it but are kind of taking a wait and see approach for the moment.
knowtheory commentedon Nov 2, 2015
@aschweigert: just as a point of note, there was a security vulnerability in the TinyMCE integration that the plugin had been using, and our solution was to just remove the wizard for the time being (it wasn't working anyway).
Would be interested if Shortcake would be worth rolling out even as a tentative feature since we currently offer no alternative.
reefdog commentedon Nov 2, 2015
Yeah, wouldn't make it a requirement for base functionality, just as a potential answer for wizardy embed option composition.