PostHog / posthog

🦔 PostHog provides open-source web & product analytics, session recording, feature flagging and A/B testing that you can self-host. Get started - free.
https://posthog.com
Other
21.86k stars 1.3k forks source link

Settings -> data attributes -> bad UX with select component #11511

Closed mariusandra closed 1 month ago

mariusandra commented 2 years ago

Bug description

First, there are two very similar looking yet different UX elements on the page, for data attributes and person display names.

The difference is that data attributes are a freeform text input, while person attributes are selected from a fixed list (via the taxonomic filter).

Perhaps we should use just one style for both cases?

Second, the UX for adding data attributes is not so good:

2022-08-26 11 53 56

At the very least we shouldn't show the dropdown autocomplete.

Environment

Additional context

Seems to come from https://github.com/posthog/posthog/pull/11253

Thank you for your bug report – we love squashing them!

clarkus commented 2 years ago

I've been working on this a bit. There are a few usage patterns I think we should consider. Each component pattern has pros and cons that make the more or less suitable based on those usage patterns.

Fields that contain a high volume of options, options that have more complex or unbreakable strings, or options that can be reordered don't work as well in the multi-select component. The multi-select has additional padding and a border - this constrains the available space and decreases the space available for selected options. We have use cases that fall under each of the patterns above.

TL;DR - we might be able to consolidate down to a single component, but we might have to bias towards specific usage patterns which could affect some real use cases we have. I'll keep working on this issue this week and will get some documentation added to the design system assets (multiple-select, labels & pills).

clarkus commented 2 years ago

I documented a bunch of multiple select usage notes here. I think if we can fix the UX issues for the data attribute component it'd be worth it. If not, we could consolidate to the free-form style component for selecting multiple values.

Frame 9817

posthog-bot commented 1 month ago

This issue hasn't seen activity in two years! If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in two weeks.

posthog-bot commented 1 month ago

This issue was closed due to lack of activity. Feel free to reopen if it's still relevant.