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

Improve UX for segment condition descriptions #4000

Closed
rolodato opened this issue May 21, 2024 · 2 comments · Fixed by #4095
Closed

Improve UX for segment condition descriptions #4000

rolodato opened this issue May 21, 2024 · 2 comments · Fixed by #4095
Assignees
Labels
improvement Improvement to the existing platform

Comments

@rolodato
Copy link
Member

Is your feature request related to a problem? Please describe.

This is the UI to create a new segment:

image

The button "Show condition descriptions" does nothing until you figure out you need to create a condition first for this to make sense. This button is also confusingly named, since the option "Show condition descriptions" looks to be disabled, but this changes to "Hide condition descriptions" once enabled:

image

I understand why this button exists, since adding a "description" field below every condition can get big fast:

image

Segment descriptions can be very valuable and we should encourage their use. We should get rid of the "Show condition descriptions" button to remove a useless choice for unfamiliar users and provide a better way to show and edit descriptions.

Describe the solution you'd like.

One alternative would be to collapse all the available actions for a condition (edit, delete, add "Or") into a collapsible menu. Example from Vercel's environment variable editor:

Screenshot 2024-05-21 at 16 51 05

This would have the added benefit of making it less likely to fat-finger the Delete button, which currently requires no confirmation and has no undo.

Describe alternatives you've considered

A quick fix might be to simply reduce the placeholder height for descriptions, and expand them as needed up to a certain limit. For example, this screenshot shows the default textbox height compared to the smallest my browser allows me to resize them:

Screenshot 2024-05-21 at 16 54 26

I would be okay with something similar to this as well - the goal here is to reduce friction for creating segments.

Additional context

Related issues:

#3920
#3918

@kyle-ssg
Copy link
Member

kyle-ssg commented Jun 4, 2024

  • The button "Show condition descriptions" does nothing until you figure out you need to create a condition

Isn't an issue once we re-add the placeholder rule.

-This button is also confusingly named, since the option "Show condition descriptions"

Ok agree yep will keep the label as show condition descriptions, I've added this in the same PR

-Segment descriptions can be very valuable and we should encourage their use

Not sure I agree here, adding descriptions for every single rule is tedious and unnecessary for simple rules and is only useful if there's nuance or complex rules.

@kyle-ssg kyle-ssg linked a pull request Jun 4, 2024 that will close this issue
5 tasks
@rolodato
Copy link
Member Author

rolodato commented Jun 4, 2024

Not sure I agree here, adding descriptions for every single rule is tedious and unnecessary for simple rules and is only useful if there's nuance or complex rules.

That's fair - we might be able to come up with a better solution later on, so I'm happy to stay this way for now.

@rolodato rolodato assigned kyle-ssg and unassigned rolodato Jun 4, 2024
@rolodato rolodato added the improvement Improvement to the existing platform label Jun 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement to the existing platform
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants