Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show filter in minimal mode if the table is filtered #2208

Merged
merged 3 commits into from
May 23, 2024

Conversation

azmy60
Copy link
Contributor

@azmy60 azmy60 commented May 16, 2024

When minimal mode is active, and there are filters in the table, we should be able to see it and clear it. To close the filter, we can do one of these:

Clear the filter manually and then click away from the input

close-filters-by-clicking-away

Click the clear button

close-filters-by-clicking-button


🚀 This description was created by Ellipsis for commit 7853ce2

Summary:

Enhances filter visibility in minimal mode by adding utility functions and updating Vue components to use these functions for consistent filter handling.

Key points:

  • Enhances filter visibility in minimal mode by adding utility functions and updating Vue components for consistent filter handling.
  • Add createTableFilter and checkEmptyFilters in utils.ts.
  • Update RowFilterBuilder.vue to handle filter visibility in minimal mode.
  • Always render RowFilterBuilder in TableTable.vue but control visibility internally.
  • Use new utility functions across components for consistent filter handling.

Generated with ❤️ by ellipsis.dev

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ Changes requested. Reviewed everything up to 19b2de1 in 1 minute and 14 seconds

More details
  • Looked at 218 lines of code in 3 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_bgXpSr05j0ifUKyE


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

22 hours left in your free trial, upgrade for $20/seat/month or contact us.

apps/studio/src/common/utils.ts Show resolved Hide resolved
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❌ Changes requested. Incremental review on 7853ce2 in 1 minute and 29 seconds

More details
  • Looked at 15 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_DljhvXzmEVmrtBLg


Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

22 hours left in your free trial, upgrade for $20/seat/month or contact us.

apps/studio/src/common/utils.ts Outdated Show resolved Hide resolved
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@rathboma rathboma merged commit e953a70 into master May 23, 2024
13 checks passed
@rathboma rathboma deleted the feat/minimal-mode branch May 23, 2024 01:40
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.

None yet

2 participants