Skip to content

Conversation

@Soare-Robert-Daniel
Copy link
Contributor

@Soare-Robert-Daniel Soare-Robert-Daniel commented Nov 3, 2025

Summary

  • Improve the positioning of the BF banner in the main dashboard page.
  • Update the visiblity condition to correctly match the BF banner.

Will affect visual aspect of the product

YES

Screenshots

CleanShot 2025-11-03 at 16 06 57@2x CleanShot 2025-11-03 at 16 07 06@2x CleanShot 2025-11-03 at 16 07 14@2x CleanShot 2025-11-03 at 16 07 22@2x CleanShot 2025-11-03 at 16 07 32@2x CleanShot 2025-11-03 at 16 07 50@2x

Test instructions

Check before Pull Request is ready:

Closes https://github.com/Codeinwp/themeisle/issues/1825

@Soare-Robert-Daniel Soare-Robert-Daniel self-assigned this Nov 3, 2025
@Soare-Robert-Daniel Soare-Robert-Daniel added the pr-checklist-skip Allow this Pull Request to skip checklist. label Nov 3, 2025
@pirate-bot pirate-bot added the pr-checklist-complete The Pull Request checklist is complete. (automatic label) label Nov 3, 2025
@pirate-bot
Copy link
Contributor

Plugin build for 95894c8 is ready 🛎️!

Note

You can preview the changes in the Playground

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a banner placeholder element with ID tsdk_banner to the support page and updates the CSS to properly style this banner while maintaining existing banner styling rules.

  • Adds a banner placeholder div with ID tsdk_banner to the support page layout
  • Updates CSS selector to exclude #tsdk_banner from the visibility:hidden rule
  • Adds specific padding style for the banner when inside .feedzy-container

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
includes/layouts/feedzy-support.php Adds tsdk_banner div element inside the feedzy-container for promotional banners
css/settings.css Updates banner visibility rules to show the new tsdk_banner and adds padding override for it

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-checklist-complete The Pull Request checklist is complete. (automatic label) pr-checklist-skip Allow this Pull Request to skip checklist.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants