Skip to content

fix: improve style of the offer overview in case of a bundle#1003

Merged
levalleux-ludo merged 3 commits intomainfrom
dapp1174-bundle-overview-style
Feb 18, 2026
Merged

fix: improve style of the offer overview in case of a bundle#1003
levalleux-ludo merged 3 commits intomainfrom
dapp1174-bundle-overview-style

Conversation

@levalleux-ludo
Copy link
Member

Description

fixes bosonprotocol/interface#1174

How to test

use storybook

BEFORE
image

AFTER
image

Copilot AI review requested due to automatic review settings February 18, 2026 14:50
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

This PR adjusts the layout of the Offer Full Description “Overview” section to improve presentation (notably for bundle offers) by making the label/value grid more responsive and preserving description formatting.

Changes:

  • Introduces a ResponsiveGridContainer wrapper around GridContainer to control grid-template-columns across breakpoints.
  • Updates the overview grids to stack on xxs and use a fixed-width first column on xs+.
  • Renders the description with white-space: pre-wrap to preserve line breaks.

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

levalleux-ludo and others added 2 commits February 18, 2026 16:29
…rFullDescription/Overview.tsx

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@levalleux-ludo levalleux-ludo merged commit 822808b into main Feb 18, 2026
9 of 11 checks passed
@levalleux-ludo levalleux-ludo deleted the dapp1174-bundle-overview-style branch February 18, 2026 16:41
@codecov
Copy link

codecov bot commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.93%. Comparing base (f6b8c7b) to head (21e8330).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1003   +/-   ##
=======================================
  Coverage   90.93%   90.93%           
=======================================
  Files         176      176           
  Lines        7247     7247           
  Branches     1355     1355           
=======================================
  Hits         6590     6590           
  Misses        657      657           
Flag Coverage Δ
common 92.59% <ø> (ø)
core-sdk 56.66% <ø> (ø)
e2e 85.71% <ø> (ø)
eth-connect-sdk 95.06% <ø> (ø)
ethers-sdk 74.45% <ø> (ø)
ipfs-storage 91.75% <ø> (ø)
metadata 94.70% <ø> (ø)
unittests 60.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

When showing the "Overview" for a bundle offer, the line breaks are not taken into account in the description

2 participants