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

Simplify Names and UI #135

Draft
wants to merge 19 commits into
base: 3.x
Choose a base branch
from
Draft

Simplify Names and UI #135

wants to merge 19 commits into from

Conversation

djay
Copy link
Member

@djay djay commented Jun 21, 2021

Currently the settings seem very technical and hard to understand what they do or how to use filters.
I've attempted to some changes to try fix some of these problems

  • rearranging fields in settings to put the most useful (and easiest to explain) items first
  • moved advanced/technical options to the end... could also move to to an advanced tab
  • rewrote much of the descriptions to use more end user understandable language (e.g. Index is not an end user understood term. neither are AND/OR logic operations)
  • changed names to reduce confusion that filters can be used for more than just collections if Tests for tiles + support contentlistingtiles without collections #133 is accepted.

Here is an example of the current state

Screen Shot 2021-06-21 at 5 54 45 pm

Name changes

Here are some of the major ones

  • "Collection Filter" -> "Result Filter"
  • "Collection Search" -> "Result Search"
  • "Collection Filter Result Sorting" -> "Result Sort"
  • "Collection Maps" -> "Filter by Map"
  • "Group by" -> "Filter by"
  • "filter group" -> "filter option"
  • "filtertype_single" -> "One Option at a time"
  • "filtertype_and" -> "Reduce Results (AND)"
  • "filtertype_or" -> "Expand Results (OR)"

TODO

  • rewrite the readme to explain how to use collectionfilter.
  • find better name. "Results" is probably not a good name, esp if you use a filter mosaic contentlistings for folder listings which aren't really search results. Maybe "Listing Filter"?
  • The combinations of Input Type, Filter Type and narrow down is still confusing to understand what you will get in the end. There must be a simpler way to present the choices
    • if you choose multiple you get checkboxes not radio or dropdowns
    • it's hard to explain what narrow down means. does narrow down when using OR even make any sense?
  • it's not very clear when a user would want to choose a different target, CSS selector or resulting listing view name.

@djay djay changed the base branch from master to collectionish June 21, 2021 10:03
@djay djay marked this pull request as draft June 21, 2021 10:05
@djay djay requested a review from petschki June 24, 2021 05:45
@djay djay marked this pull request as ready for review June 24, 2021 15:00
Base automatically changed from collectionish to master July 19, 2021 11:46
@JeffersonBledsoe JeffersonBledsoe marked this pull request as draft September 21, 2021 09:22
@petschki petschki changed the base branch from main to 3.x August 2, 2023 12:45
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.

4 participants