Move the global media settings to site configuration. The content type settings should be made available with the respective node type (admin/content/node-type/*) if possible.

Comments

jmstacey’s picture

jmstacey commented
Title: Menu item locations » Revise menu item locations
Assigned: Unassigned » jmstacey
Status: Active » Fixed

Commit #216730

* Global settings moved to site configuration (admin/settings).
* Content type settings moved to the content type edit page as a new fieldset (admin/content/node-type/*).
* Code refactoring: media_admin.inc renamed to media_settings.inc.

aaron’s picture

aaron commented

Looks good so far. We'll need to either ensure there's a working default (since the node type forms still reference that), or remove the idea of a default. I could go either way with that, and am interested in hearing other opinions (unless you have a strong idea on that). Maybe we could poll users at #drupal-dojo.

Great work, you're off to a strong start!

aaron’s picture

aaron commented

If we do keep a set of working media defaults, we would probably want to add some js behaviors to the node type form so that everything after the first checkbox is initially grayed out, and only becomes active when checking the override check box (and vice versa).

aaron’s picture

aaron commented
Status: Fixed » Needs work
aaron’s picture

aaron commented
Status: Needs work » Fixed

fixed

Status: Fixed » Closed (fixed)
Issue tags: -gsoc, -gsoc2009, -gsoc2009-jmstacey

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

Issue tags: +gsoc, +gsoc2009, +gsoc2009-jmstacey

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.