Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
It makes sense to me if the background image is set in the same place as one sets the logo and the favicon, which means the theme settings page under admin/appearance. HOOK_form_system_theme_settings_alter() would probably need to be implemented.
Comments
Comment #1
Les LimLooks like Background module did just this for D6, but there's no port to D7.
http://drupal.org/project/background
Comment #2
drclaw CreditAttribution: drclaw commentedI'm don't think I agree. Putting the background image form on the theme settings page would restrict the user to only being able to globally apply a background image to the entire site. You can achieve the same effect right now (with the additional bg_image_context or bg_image_ui modules) but you also have the flexibility to apply background images only to certain pages as well.
Comment #3
Les LimI probably shouldn't have worded it as "moving" the functionality; what I was really talking about was having the additional option of setting a default background image for the theme from the theme settings page. bg_image_context or bg_image_ui would be able to override the default. The main reason I'd find this useful is in the case where a user just wants to be able to switch out the background image globally, without requiring the Context module dependency.
This would probably be a new submodule, which I'd be willing to work on if you agree. If you still don't find think this is necessary, close the issue again and I'll drop it. :)
Comment #4
drclaw CreditAttribution: drclaw commentedWell, if you feel strongly enough about it, I'll definitely accept a patch.
How about we put it right in the bg_image module? Since it doesn't really do anything out-of-the-box we may as well at least give it this, right?
Comment #5
drclaw CreditAttribution: drclaw commentedGonna close this one down for now as it's been a pretty long time of no activity