-
Notifications
You must be signed in to change notification settings - Fork 62
Description
@bryanwillis brings up that by switching from genesis_footer_creds_text to genesis_footer_output we gain more control over modifying the footer area.
See PR #110
My first instinct is as a starter theme, we should stick with genesis_footer_creds_text since we're only changing the text with the understanding that others modifying the theme will most likely use genesis_footer_output because they'll want to make more changes.
As I think about Bryan's point, if I'm modifying the theme it would be easier to already have the filter I want to use and just modify it rather than having to determine the filter I want is genesis_footer_output.
In addition to the filter change, this PR centers the Footer Credentials, which is not an option when using genesis_footer_creds_text.
I am a little concerned about the edge case where someone is already applying a filter to genesis_footer_creds_text and then they update. I'd hate to see their Footer Credentials revert to the project Footer Credentials.
Maybe we should keep genesis_footer_creds_text and add a second call which goes to genesis_footer_output and adds the <p class="text-center"> tag around it, thought perhaps this is over-engineering.
Okay, this ended up being a brain dump. Let's give this a little thought and decide how to handle it. I'd love to hear input if others have thoughts.