Skip to content

Conversation

@lyubov-voloshko
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings January 29, 2026 12:22
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 fixes visual and layout issues in the table view components by improving dark mode support for filter chips and enhancing the row preview sidebar's header behavior.

Changes:

  • Added dark mode styling for filter chips to improve visibility with appropriate dark background colors and white text
  • Increased bottom margin of table surface to provide more spacing
  • Made the row preview sidebar header sticky and removed unnecessary spacing elements

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
frontend/src/app/components/dashboard/db-table-view/saved-filters-panel/saved-filters-panel.component.css Updated static filter chip colors for dark mode from light gray to dark blue-gray with proper text contrast
frontend/src/app/components/dashboard/db-table-view/db-table-view.component.css Added dark mode styles for active filter chips and increased table-surface margin-bottom from 24px to 72px
frontend/src/app/components/dashboard/db-table-view/db-table-row-view/db-table-row-view.component.html Removed unnecessary <br /> tag for cleaner spacing
frontend/src/app/components/dashboard/db-table-view/db-table-row-view/db-table-row-view.component.css Made sidebar header sticky with proper positioning, z-index, and padding adjustments

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

@lyubov-voloshko lyubov-voloshko merged commit 2f2777c into main Jan 29, 2026
17 checks passed
@lyubov-voloshko lyubov-voloshko deleted the table-view-fixes branch January 29, 2026 13:00
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