I have no use for a copyright line, but I really want to display a little unobtrusive "Powered by [link]Drupal[/link]" line where the copyright line is located in this theme. There's a setting for editing the name of the copyright holder, however it's unnecessarily narrow in that it doesn't allow me to change the "Copyright (c) [year]" part.
I think the setting would be more useful if it covered the whole line instead.
P.S.: The spelling really is "copyright", as in who's got the rights to copy the contents. The incorrect spelling "copywrite" could be corrected in user-visible strings as well and if the variable is repurposed in an update to fit the whole line, maybe it can be renamed to the correct spelling as well - in fact, you could have an easy migration path by defaulting a 'copyright' variable to the following expression:
t('Copyright !c !year, !name', array('!c' => '©', '!year' => date('Y'), '!name' => theme_get_setting('copywrite_holder')))
but using theme_get_setting('copyright') instead if it's not empty.
Comment | File | Size | Author |
---|---|---|---|
#6 | settings.png | 26.14 KB | mheinke |
#6 | front-footer.png | 238.09 KB | mheinke |
#5 | theme-settings-bug-patch-1.patch | 3.84 KB | mheinke |
Comments
Comment #1
mheinke CreditAttribution: mheinke commentedill put a patch together for this.
thanks!
Comment #2
mheinke CreditAttribution: mheinke commentedtest this patch
Comment #3
mheinke CreditAttribution: mheinke commentedComment #4
jpetso CreditAttribution: jpetso commentedThanks, I'll have a look later. (No time right now, but I'll get back to you.) On a quick glance, I saw that there's a 'copright_information' field in there somewhere, that's likely to be a typo, but I haven't had the time to figure out whether or how it impacts the functionality of this patch.
Comment #5
mheinke CreditAttribution: mheinke commentedComment #6
mheinke CreditAttribution: mheinke commentedtested with simplytest.me (2.x branch + latest patch.) here are screenshots
Comment #7
mheinke CreditAttribution: mheinke commentedthis patch has been applied to the 2.x-dev branch and will be in the next release
Comment #8
mheinke CreditAttribution: mheinke commentedComment #9
jpetso CreditAttribution: jpetso commentedSorry for the late response, went travelling for a few weeks. The new patch works great. Thanks!
Comment #10
mheinke CreditAttribution: mheinke commentedno worries! thanks for confirming!
Comment #11
mheinke CreditAttribution: mheinke commentedlet me know if there are any other features you want :)
Comment #12
mheinke CreditAttribution: mheinke commented