Skip to content

Top Sidebar Banner#21

Merged
jasperf merged 1 commit into
mainfrom
top-sidebar-banner
May 26, 2025
Merged

Top Sidebar Banner#21
jasperf merged 1 commit into
mainfrom
top-sidebar-banner

Conversation

@jasperf

@jasperf jasperf commented May 26, 2025

Copy link
Copy Markdown
Contributor

This pull request updates the Banner Container Plugin to version 1.7.0, introducing a breaking change to the sidebar banner location. The change replaces the get_sidebar hook with the dynamic_sidebar_before hook for improved positioning and theme compatibility. Additionally, the update includes automatic migration of legacy settings, UI enhancements, and updated documentation.

Breaking Change: Sidebar Banner Location Update

  • Replaced the get_sidebar hook with dynamic_sidebar_before for sidebar banners, ensuring banners display above all sidebar widgets for better positioning. (includes/class-iwz-banner-container-settings.php, includes/class-iwz-banner-container.php, README.md, CHANGELOG.md) [1] [2] [3] [4]
  • Updated logic and settings to support the new hook, including migrating legacy settings and ensuring compatibility with themes using dynamic_sidebar(). (includes/class-iwz-banner-container-settings.php, includes/class-iwz-banner-container.php) [1] [2] [3] [4] [5]

Documentation Updates

  • Updated the README to explain the new sidebar banner behavior and its advantages, including better positioning and theme compatibility. (README.md) [1] [2]
  • Added a note in the admin settings UI to clarify the new behavior of sidebar banners. (includes/class-iwz-banner-container-settings.php)

Version and Changelog

  • Bumped the plugin version to 1.7.0 and updated the changelog to document the changes. (iwz-banner-container-plugin.php, CHANGELOG.md) [1] [2] [3]

@jasperf jasperf merged commit e8f7946 into main May 26, 2025
5 checks passed
@jasperf jasperf deleted the top-sidebar-banner branch May 26, 2025 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant