I'm not sure what the purpose of having the embed code in a separate menu item, outside of the admin menu, but even if it stays there (I don't think it should- I hate when modules automatically add themselves to the top level navigation menu) I think you should provide it on the admin page. Two places where it would be good are on the widget preview page and via a link under "Widgets" at admin->build->embed-widgets.

Comments

jtsnow’s picture

jtsnow commented

The "Embed Widgets" top-level menu item was there in the D5 version. It's purpose, and why I have left it there until now, is that it makes it easy for non-admins to get widget code. The user interface will be much better by the time I'm done with it. I also dislike modules that add top-level navigation items, but a page where end users can customize widgets and grab the code will be necessary.

A menu item link may not be necessary if "embed" links are added to every page, block, and view, that have been added as sources. I'm not sure which is less intrusive, but I plan on making an "Embed Widgets Settings" page under Site Configuration to manage these sort of options.

As for now, the link has been removed and the embed code is available in the admin section on the widget preview page.

aaron’s picture

aaron commented

I like your reasoning, and see both sides of this issue. I think this would be a perfect place for MENU_SUGGESTED_ITEM.

aaron’s picture

aaron commented

bonus points if the admin can turn the menu on/off from an embed widgets settings page, instead of having to somehow magically know the suggested menu item is available. at the least, there should be a mention of it in the resulting documentation.

alex ua’s picture

alex ua commented
Title: Make Embed Code Available in the Embed Code Admin Section » Make Embed Code Available in the Embed Widget Admin Section

I didn't mean to imply that the embed code should be available only on the Embed Widget admin section, just that it should it should be there as well.

alex ua’s picture

alex ua commented
Status: Active » Fixed

I'm going to go ahead and mark this as fixed, since the widget embed code is now on the admin page. We should open up a new issue in the near future to discuss where else the code should be available. (For example, I'd love a Views setting, argument, or something else that would allow you to have embed widget code on every page of the view, similarly to the Feed Selector argument)

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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