Skip to content

fix: restore dark theme background on column validations panel - #1196

Merged
chavda-bhavik merged 1 commit into
nextfrom
fix/column-validations-panel-bg
Sep 1, 2026
Merged

chavda-bhavik merged 1 commit into
nextfrom
fix/column-validations-panel-bg

Conversation

@chavda-bhavik

Copy link
Copy Markdown
Member

Summary

  • Follow-up to feat: enforce configurable max image size and improve column management flow #1195: extracting the GatedField component dropped the colors import in ColumnForm.tsx and left a stray light-gray placeholder (#f8f9fa) as the "Column Validations" panel's background instead of the app's dark colors.BGSecondaryDark (#1D1E22), which clashed visibly with the rest of the (dark-themed) modal.
  • Restores the correct background.

Test plan

  • apps/web type-checks cleanly (tsc --noEmit)
  • Open the column add/edit modal on quality and confirm the right-hand "Column Validations" panel matches the app's dark theme

Co-Authored-By: Claude Sonnet 5 noreply@anthropic.com

https://claude.ai/code/session_01Hqy3rtwmRHKRsqKY5ntuT9

The GatedField extraction dropped the colors import and left a
placeholder light-gray fallback on the Column Validations panel,
which clashed with the app's dark UI instead of using the same
BGSecondaryDark shade as the rest of the modal.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Hqy3rtwmRHKRsqKY5ntuT9
@nx-cloud

nx-cloud Bot commented Sep 1, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit c2e1fcb

Command Status Duration Result
nx run-many --target=build --all ✅ Succeeded 1m 4s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-09-01 10:54:01 UTC

@chavda-bhavik chavda-bhavik self-assigned this Sep 1, 2026
@chavda-bhavik
chavda-bhavik merged commit 006648f into next Sep 1, 2026
2 checks passed

This branch was successfully deployed

1 active deployment
Development — c2e1fcb6 Deployed Sep 1, 2026 by chavda-bhavik via build #1634
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant