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

Hide option Remove all filters #14710

Open
wants to merge 1 commit into
base: 3.x
Choose a base branch
from

Conversation

amoradel
Copy link

@amoradel amoradel commented Nov 5, 2024

Description

Evaluate if all filter indicators can be removed; hide option to remove all filters if none are removable $indicator->isRemovable() = false

Visual changes

Hide option.
image

Show option.
image

Functional changes

  • Code style has been fixed by running the composer cs command.
  • Changes have been tested to not break existing functionality.
  • Documentation is up-to-date.

…ve all filters if none are removable [$indicator->isRemovable() = false]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant