feat: improve clear override UX with feature-flagged options #566
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Implements two new UX options for the clear override action, gated behind LaunchDarkly feature flags: Option A (clear-override-hover-reveal): Compact badge that expands on hover to show 'Clear' text, making the action more discoverable while preserving space when not interacting. Option C (clear-override-icon-button): Replaces the small dot with a recognizable clear/X icon button that is universally understood as a remove action. Both options are disabled by default, with the original dot indicator remaining as the fallback. Addresses REL-11957 Co-Authored-By: Ramon Niebla <rniebla@launchdarkly.com>
Contributor
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Replaces two boolean flags with a single multivariate flag 'clear-override-ux-variant' that supports traffic splitting between: - 'default': Original dot indicator - 'hover-reveal': Expands on hover to show 'Clear' text - 'icon-button': Recognizable X icon button This makes it easier to run experiments and split traffic between variants. Co-Authored-By: Ramon Niebla <rniebla@launchdarkly.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
What does this PR do?
Addresses REL-11957 - Improves the "clear override" UX which was reported as challenging to use (the small cyan dot wasn't obviously clickable).
This PR implements two alternative UX options, gated behind LaunchDarkly feature flags in the
developer-toolbarproject:Option A - Hover Reveal (
clear-override-hover-reveal): The indicator expands on hover to show "Clear" text, making the action more discoverable while preserving space when not interacting.Option C - Icon Button (
clear-override-icon-button): Replaces the small dot with a recognizable clear/X icon button that is universally understood as a remove action.Both flags default to
false, keeping the original dot indicator as the fallback behavior.Type of Change
Human Review Checklist
clear-override-hover-reveal,clear-override-icon-button)Screenshots/Videos
Screenshots not available - enable the feature flags in the
developer-toolbarproject to test the new UX options.Link to Devin run: https://app.devin.ai/sessions/ba2f0534579942989f3a85721e0ca681
Requested by: Ramon Niebla (@nieblara)