Skip to content

Add Nitro ad slots to AuctionHouse and Bazaar price graphs#1476

Merged
matthias-luger merged 2 commits intodevelopfrom
add-add-to-item-page
Mar 4, 2026
Merged

Add Nitro ad slots to AuctionHouse and Bazaar price graphs#1476
matthias-luger merged 2 commits intodevelopfrom
add-add-to-item-page

Conversation

@matthias-luger
Copy link
Collaborator

No description provided.

Copy link
Contributor

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

Adds a Nitro ad placement below the item price graphs, and adjusts graph layout sizing to make room for the ad on the item page.

Changes:

  • Add a NitroAdSlot below the Auction House price graph.
  • Add a NitroAdSlot below the Bazaar price graph.
  • Reduce Bazaar graph wrapper heights to accommodate the new below-graph content.

Reviewed changes

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

File Description
components/PriceGraph/BazaarPriceGraph/BazaarPriceGraph.tsx Adds a Nitro ad slot below the bazaar graph UI.
components/PriceGraph/BazaarPriceGraph/BazaarPriceGraph.module.css Reduces chart wrapper heights to preserve page layout with the new ad slot.
components/PriceGraph/AuctionHousePriceGraph/AuctionHousePriceGraph.tsx Adds a Nitro ad slot below the auction graph and reshapes the “Avg Price / actions” header area.

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

You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@matthias-luger matthias-luger merged commit 6fd3cf3 into develop Mar 4, 2026
1 check passed
@matthias-luger matthias-luger deleted the add-add-to-item-page branch March 4, 2026 16:05
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.

2 participants