feat: compare empty string display

[Zaimwa9]

Thanks for submitting a PR! Please check the boxes below:

• I have read the Contributing Guide.
• I have added information to docs/ if required so people know about the feature.
• I have filled in the "Changes" section below.
• I have filled in the "How did you test this code" section below.

Changes

• Display "" for empty values in compare view

How did you test this code?

• Against staging

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

_{Tip: disable this comment in your organization's Code Review settings.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
flagsmith-frontend-preview	Ready	Preview, Comment	May 12, 2026 4:09pm
flagsmith-frontend-staging	Ready	Preview, Comment	May 12, 2026 4:09pm

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Ignored	Preview	May 12, 2026 4:09pm

[github-actions]

Docker builds report

Image	Build Status	Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-7498	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-7498	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-7498	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-api-test:pr-7498	Finished ✅	Skipped
ghcr.io/flagsmith/flagsmith-api:pr-7498	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith:pr-7498	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith:pr-7498	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-private-cloud:pr-7498	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-api:pr-7498	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-frontend:pr-7498	Finished ✅	Results ✅
ghcr.io/flagsmith/flagsmith-frontend:pr-7498	Finished ✅	Results ✅

[github-actions]

Playwright Test Results (oss - depot-ubuntu-latest-16)

 1 passed

Details

 1 test across 1 suite
 33.3 seconds
 e041dd7
 🔄 Run: #16707 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

 1 passed

Details

 1 test across 1 suite
 43.5 seconds
 e041dd7
 🔄 Run: #16707 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

 1 passed

Details

 1 test across 1 suite
 40.8 seconds
 e041dd7
 🔄 Run: #16707 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

 3 passed

Details

 3 tests across 3 suites
 32.4 seconds
 e041dd7
 🔄 Run: #16707 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

 1 passed

Details

 1 test across 1 suite
 33.1 seconds
 edf9b9c
 🔄 Run: #16710 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

 18 passed

Details

 18 tests across 14 suites
 1 minute, 9 seconds
 edf9b9c
 🔄 Run: #16710 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

 1 passed

Details

 1 test across 1 suite
 34.6 seconds
 edf9b9c
 🔄 Run: #16709 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

 3 passed

Details

 3 tests across 3 suites
 49.1 seconds
 edf9b9c
 🔄 Run: #16709 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

 3 passed

Details

 3 tests across 3 suites
 1 minute, 10 seconds
 edf9b9c
 🔄 Run: #16709 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

 3 passed

Details

 3 tests across 3 suites
 37 seconds
 edf9b9c
 🔄 Run: #16710 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

 1 passed

Details

 1 test across 1 suite
 55.8 seconds
 edf9b9c
 🔄 Run: #16709 (attempt 1)

[github-actions]

Visual Regression

16 screenshots compared. See report for details.
View full report

[talissoncosta]

Looks good. This is a much cleaner solution.

Quick question: would there be a strong reason not to make this the default everywhere rather than gating it behind isCompareEnv?

The original noise concern was about the [no value] chip rendering on every null row, which we've dropped. "" is rare enough that always rendering it shouldn't add meaningful noise to the features list — and it'd solve the same disambiguation problem on any future surface without per-caller opt-in.

Just raising as a discussion point, not blocking.

[Zaimwa9]

@talissoncosta added it in edf9b9c

Let me know your thoughts