Contrib.social

Change unclear Exposed filter option "Limit list to selected items" title and description

Open on Drupal.org →
Created on 5 May 2020, almost 5 years ago
Updated 26 January 2025, 3 months ago

Hi!

This request is a natural follow-up of an existing D8 issue:
💬 Using "Limit list to selected items" on exposed filters not working properly Active
And D7 issue:
#510068: Using "Limit list to selected items" on exposed filters not working →

The title and description of the checkbox: "Limit list to selected items" / "If checked, the only items presented to the user will be the ones selected here." is not clear in that it only applies to the filter options and not the view results.

The title and description could be updated to clarify this:

Title: "Limit filter list to selected items"

Description: "If checked, the only filter options presented to the user will be the ones selected here. This checkbox has no effect on view results list."

And to clarify checkbox purpose completely, and to help those who need results of selected-items-only to be inside view-results-list as well - we could add some documentation words to description:

"If you need view results list to contain results of selected items only, please create additional (same, but non-exposed) filter."

Precise checkbox description would mean less "Closed as designed" Drupal Core issues in the future.

📌 Task
Status

Needs work

Version

11.0 🔥

Component

views.module

Created by

🇭🇷Croatia devad

Live updates comments and jobs are added and updated live.
  • Needs manual testing

    The change/bugfix cannot be fully demonstrated by automated testing, and thus requires manual testing in a variety of environments.

  • Needs usability review

    Used to alert the usability topic maintainer(s) that an issue significantly affects (or has the potential to affect) the usability of Drupal, and their signoff is needed. When adding this tag, make it easy to review the issue. Make sure the issue summary describes the problem and the proposed solution. Screenshots usually help a lot! To get sign-off on issues with the "Needs usability review" tag, post about them in the #ux channel on Drupal Slack, and/or attend a UX meeting to demo the patch and get direct feedback from designers/UX folks/product management on next steps. If an issue represents a significant new feature, UI change, or change to the general "user experience" of Drupal, use Needs product manager review instead. See the scope of responsibilities for product managers.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment 3 months ago →
    🇳🇱Netherlands johnv
  • Comment 2 months ago →
    🇳🇱Netherlands johnv

    As a summary of the above discussion:
    #1-#34 are about the new text proposal
    #28 contains the latest text proposal
    #39-#44 are about other proposal that better deserves its own issue, if still needed.
    #40 expresses some thoughts about current behaviour
    I tested in D11.1 the behaviour of the (exposed) filter operators and options: Whatever the limit/expose settings, the values that are set in 'Filter criterion' form, are directly applied to the view, when a user calls the view.

    Issue summary is updated to standard format.
    Issue moved to views_ui.module.

    Regarding the text proposal, I find the additional lines hard to understand. Or even: I do not understand them at all.

    As an alternative, let us compare with the other option:
    [ ] "Limit the available operators" = "Limit the available operators to be shown on the exposed filter."
    Then 'our' option would just be:
    [ ] "Limit the available options" = "Limit the available options to be shown on the exposed filter to the ones selected above."
    This is what is proposed in attached patch.

  • First commit to issue fork.
  • Merge request !11132Change unclear Exposed filter option "Limit list to selected items" title and description → (Open) created by VladimirAus
  • Pipeline finished with Failed
    2 months ago
    Total: 414s
    #416833
  • Comment 2 months ago →
    🇺🇸United States smustgrave

    Just FYI UI changes should include screenshots in the summary.

  • Comment 2 months ago →
    🇭🇷Croatia devad

    Re: #51-#54

    Cosmetic change of title and description is not what this issue is meant for originally . The new description should warn users of existing bug behavior. Please read previous conversation in this issue. Comment #40 in particular but other comments as well.

  • Comment 2 months ago →
    🇺🇸United States smustgrave

    yea seems like there was still some discussion going on, didn't read the comments yesterday hence never changed the status.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024