Skip to content

fix(sidebar): prevent clear button (X) from overlapping scrollbar in table filter#403

Merged
debba merged 1 commit into
TabularisDB:mainfrom
viniciusmelocodes:fix-layout-input-filter
Jul 1, 2026
Merged

fix(sidebar): prevent clear button (X) from overlapping scrollbar in table filter#403
debba merged 1 commit into
TabularisDB:mainfrom
viniciusmelocodes:fix-layout-input-filter

Conversation

@viniciusmelocodes

Copy link
Copy Markdown
Contributor
  • Increased right padding on filter inputs (pr-10)
  • Adjusted X button position (right-3 + vertical centering)
  • Applied consistently across ExplorerSidebar, SidebarSchemaItem and SidebarDatabaseItem

…table filter

- Increased right padding on filter inputs (`pr-10`)
- Adjusted X button position (`right-3` + vertical centering)
- Applied consistently across ExplorerSidebar, SidebarSchemaItem and SidebarDatabaseItem
@kilo-code-bot

kilo-code-bot Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • src/components/layout/ExplorerSidebar.tsx
  • src/components/layout/sidebar/SidebarDatabaseItem.tsx
  • src/components/layout/sidebar/SidebarSchemaItem.tsx

Reviewed by laguna-m.1-20260312:free · Input: 174.5K · Output: 3.3K · Cached: 6.5K

@debba

debba commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Looks good to me, thanks Vinícius! I'll merge this shortly. These UI polish fixes really do make a difference, so please keep them coming.

By the way, if you're up for it, come hang out with us on Discord: https://discord.com/invite/K2hmhfHRSt

@debba debba merged commit d15d294 into TabularisDB:main Jul 1, 2026
1 check passed
@viniciusmelocodes

Copy link
Copy Markdown
Contributor Author

Looks good to me, thanks Vinícius! I'll merge this shortly. These UI polish fixes really do make a difference, so please keep them coming.

By the way, if you're up for it, come hang out with us on Discord: https://discord.com/invite/K2hmhfHRSt

I'm already part of the Tabularis Discord group. Thanks for the new invitation.

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