Skip to content

Conversation

@DorianMaliszewski
Copy link
Member

Summary

Type

  • Bug

Summarize concisely:

What is expected?

(Description of the new behavior)
Filter out invalid child as they will not be rendered it caused next cells to get the wrong style (move to the right) because we use index to retrieve column style

The following changes were made:

(Describe what you did)

  1. Filter out bad children in List.Row render

  2. Add a key on column provider

Relevant logs and/or screenshots

Page Before After
url screenshot screenshot
url screenshot screenshot
@DorianMaliszewski DorianMaliszewski self-assigned this Dec 11, 2025
@DorianMaliszewski DorianMaliszewski added the bug Something isn't working label Dec 11, 2025
@changeset-bot
Copy link

changeset-bot bot commented Dec 11, 2025

🦋 Changeset detected

Latest commit: 6140cf1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
@ultraviolet/ui Minor
@ultraviolet/form Patch
@ultraviolet/nextjs Patch
@ultraviolet/plus Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov
Copy link

codecov bot commented Dec 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.58%. Comparing base (c6de62e) to head (6140cf1).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@ Coverage Diff @@ ## main #5878 +/- ## ========================================== + Coverage 91.75% 92.58% +0.82%  ========================================== Files 335 302 -33 Lines 7073 6526 -547 Branches 2734 2549 -185 ========================================== - Hits 6490 6042 -448  + Misses 583 484 -99 
Files with missing lines Coverage Δ
packages/ui/src/components/List/Row.tsx 88.40% <100.00%> (+0.34%) ⬆️

... and 33 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c6de62e...6140cf1. Read the comment docs.

🚀 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.
@DorianMaliszewski DorianMaliszewski enabled auto-merge (squash) December 11, 2025 13:50
@DorianMaliszewski DorianMaliszewski merged commit 22e27d6 into main Dec 11, 2025
17 checks passed
@DorianMaliszewski DorianMaliszewski deleted the fix/column-wrongly-sized-offer-list branch December 11, 2025 13:52
@scaleway-bot scaleway-bot mentioned this pull request Dec 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

5 participants