fix: Allow toggling dedupe off + update save notification#3286
fix: Allow toggling dedupe off + update save notification#3286
Conversation
e151f8d to
f108eb5
Compare
tw4l
left a comment
There was a problem hiding this comment.
Tested locally and against dev, worked as expected in both places and makes for a nice experience.
emma-sg
left a comment
There was a problem hiding this comment.
Seeing some issues with rapid changes, as well as with the popover appearing over an interactive area, which is a little annoying. Could we move it above the item maybe, rather than to the left of it?
Screen.Recording.2026-05-04.at.7.17.54.PM.mov
This should be fixed now.
I think the tooltip gets lost when it's above the item, testing on a collection with multiple workflows. Updated to place the tooltip on the right for large screens and decreased the duration to 4 seconds. |
emma-sg
left a comment
There was a problem hiding this comment.
Nice! The one other odd thing I'm seeing is that it seems to animate to/from the wrong place when flipped to the left on smaller screens, I'm not sure if this is an upstream bug in shoelace though
Related to #3253
Changes
Manual testing
Screenshots
Follow-ups
A common component or render template should be created if we continue to follow this pattern. As a part of that work, the success status should be light-themed to match the success and info toast messages.