You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If I'm understanding correctly, indieweb-publisher uses a universal indieweb_publisher_option() function to check each option from the customizer
This replaces multiple functions that indepenent publisher has for each option . e.g. independent_publisher_use_single_column_layout()
The problem is that indieweb_publisher_option only checks one option: "indieweb_publisher_general_options"
But some of the theme settings are saved in other options:
indieweb_publisher_excerpt_options
indieweb_publisher_layout_options
Two solution ideas:
Merge all of the options into one field, either "indieweb_publisher_general_options" or simple "indieweb_publisher_options".
Pros: Simple
Cons: Potential for confusion if pulling in updates from independent publisher
Create three functions to capture all three fields
indieweb_publisher_general_option
indieweb_publisher_excerpt_option
indieweb_publisher_layout_option
Each of these functions could check a specific option field.
Pros: Maintain consistence with independent publisher
Cons: A bit more work. Loses some of the clarity of single function solution
The text was updated successfully, but these errors were encountered:
Maybe related to #13
If I'm understanding correctly, indieweb-publisher uses a universal
indieweb_publisher_option()
function to check each option from the customizerThis replaces multiple functions that indepenent publisher has for each option . e.g.
independent_publisher_use_single_column_layout()
The problem is that
indieweb_publisher_option
only checks one option: "indieweb_publisher_general_options"But some of the theme settings are saved in other options:
Two solution ideas:
Merge all of the options into one field, either "indieweb_publisher_general_options" or simple "indieweb_publisher_options".
Pros: Simple
Cons: Potential for confusion if pulling in updates from independent publisher
Create three functions to capture all three fields
Each of these functions could check a specific option field.
Pros: Maintain consistence with independent publisher
Cons: A bit more work. Loses some of the clarity of single function solution
The text was updated successfully, but these errors were encountered: