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

Endlessly scrolling property filter #5962

Closed
mariusandra opened this issue Sep 15, 2021 · 13 comments
Closed

Endlessly scrolling property filter #5962

mariusandra opened this issue Sep 15, 2021 · 13 comments
Labels
enhancement New feature or request

Comments

@mariusandra
Copy link
Collaborator

Is your feature request related to a problem?

This tool is great:

image

But with a lot of tabs and a few scattered results, it can get hard to find things:

image

image

Describe the solution you'd like

I'd like an endless list, which would dynamically update the tabs. Basically the four infinitely scrolling lists should be combined into one, with one extra item between sets to denote a new category.

image

If I click the tabs, the list just scrolls to the right location. Clicking tabs with "0" does nothing.

Describe alternatives you've considered

Clicking though all the tabs to find what I want.

Additional context

This will get more needed when we add other tabs to the filter. For example to separate posthog and other events, or when adding support for groups.

Thank you for your feature request – we love each and every one!

@mariusandra mariusandra added the enhancement New feature or request label Sep 15, 2021
@Twixes
Copy link
Member

Twixes commented Sep 15, 2021

This used to be the case, right? But we moved to tabs to make handling lots of options easier, while clarifying different filtering types. I don't think it's a good idea to keep tabs AND combine them in the search list at the same time, as then the separation becomes really confusing.

@paolodamico
Copy link
Contributor

I think the right solution is #5381 and allowing users to switch with "tab" hotkey (or other simple key), thoughts? CC @clarkus

@clarkus
Copy link
Contributor

clarkus commented Nov 29, 2021

There are other considerations here - group analytics is adding 1–5 extra items to this component. I think we're reaching the limit of tabs and need to explore other options here. Being able to switch easily makes this a bit less painful, but the component needs to be updated to account for the higher volume of events and the higher volume of categories / tabs.

Related discussion at #7073 (comment)

@clarkus
Copy link
Contributor

clarkus commented Nov 30, 2021

Here's a concept for a flattened property list. The thing that's missing is the tab navigation that lets you quickly jump to a category.

This idea be very reliant on, related to:

Property list - flat

https://www.figma.com/file/gQBj9YnNgD8YW4nBwCVLZf/PostHog-App?node-id=5573%3A33304

@clarkus
Copy link
Contributor

clarkus commented Dec 1, 2021

@mariusandra here's an alternative to tabs. As we discussed, it's a multi-line set of pills that summarizes the matching result count for each category. When there are 0 matching results, the category is grayed out and isn't actionable. When there is a matching result, the category can be clicked on to jump to that specific area of the list.

Screen Shot 2021-12-01 at 2 50 01 PM

https://www.figma.com/file/gQBj9YnNgD8YW4nBwCVLZf?node-id=5597:35546#129608055

@mariusandra
Copy link
Collaborator Author

Beautiful! Assuming implementing a good multi-category vertical scroll will take a bit of effort, is there a quick change we can do to highlight one pill and show one scrolling list below?

@clarkus
Copy link
Contributor

clarkus commented Dec 1, 2021

Beautiful! Assuming implementing a good multi-category vertical scroll will take a bit of effort, is there a quick change we can do to highlight one pill and show one scrolling list below?

Sure here are two ideas - I prefer the first option, but would want to balance the prominence of the pills with any inline actions shown in the list. Also updated in figma.

Screen Shot 2021-12-01 at 3 51 37 PM

@paolodamico
Copy link
Contributor

Love this approach!!! As a follow-up (separate issue), can we add the taxonomy (description, tags, ...) to events where available? I imagine it would appear when hovering as we have for PostHog events and actions?

@paolodamico
Copy link
Contributor

With the new pills design, and implementation, can we close this @mariusandra @clarkus ?

@mariusandra
Copy link
Collaborator Author

The idea was still to have the list scroll endlessly as well. The pills already solve a big discoverability problem, but it'd still be faster to scroll through all the lists in one big list, especially if you have < 50 items overall.

@mariusandra
Copy link
Collaborator Author

@paolodamico @clarkus do you think the current implementation with pills + tabs is good enough?

2021-12-14 10 57 59

We don't have the infinite scroll through all tabs. This means when you scroll to the end of "event properties" you won't automatically see a new list for "person properties", and so on.

We could add that, but it'll require a significant refactor (read: day of work?). Do we:

a) prioritise this now
b) pause until next year
c) close this issue, consider the current implementation good enough, and revisit once someone feels like we need an upgrade (e.g. users complain)

I'd go for C personally, though see some value in A.

@clarkus
Copy link
Contributor

clarkus commented Dec 14, 2021

I agree we should go for C. I think this is a substantial enough improvement to close out for now and watch usage.

@paolodamico
Copy link
Contributor

Perfect, let's do it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants