style: use group opacity for text selection to fix overlapping artifacts#20564
style: use group opacity for text selection to fix overlapping artifacts#20564kairosci wants to merge 6 commits intomozilla:masterfrom
Conversation
|
/botio preview |
From: Bot.io (Linux m4)ReceivedCommand cmd_preview from @nicolo-ribaudo received. Current queue size: 0 Live output at: http://54.241.84.105:8877/ec4a36296087ca7/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_preview from @nicolo-ribaudo received. Current queue size: 0 Live output at: http://54.193.163.58:8877/229f1fcad949cbc/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/ec4a36296087ca7/output.txt Total script time: 1.00 mins Published |
From: Bot.io (Windows)SuccessFull output at http://54.193.163.58:8877/229f1fcad949cbc/output.txt Total script time: 3.51 mins Published |
|
/botio preview |
From: Bot.io (Linux m4)ReceivedCommand cmd_preview from @nicolo-ribaudo received. Current queue size: 0 Live output at: http://54.241.84.105:8877/9c5be3bf74e6c36/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_preview from @nicolo-ribaudo received. Current queue size: 0 Live output at: http://54.193.163.58:8877/deadfac15add5fc/output.txt |
From: Bot.io (Linux m4)SuccessFull output at http://54.241.84.105:8877/9c5be3bf74e6c36/output.txt Total script time: 1.01 mins Published |
From: Bot.io (Windows)SuccessFull output at http://54.193.163.58:8877/deadfac15add5fc/output.txt Total script time: 3.31 mins Published |
|
Basically you're reverting a part of what it has been done in: There are several problems with this patch:
I don't understand how it's possible to miss the floating toolbar being barely visible... It's the first thing I saw...
So I've the feeling you're using a LLM or something like that: I don't have any concern with that but some LLM are just tools here to help us but not to do the job. Note that I take the time to write this answer, so try to make a good use of your time and ours ! thank you. |
5155dd8 to
f68b43a
Compare
f68b43a to
7a9e8e3
Compare
|
Unfortunately in that second screenshot the tooltip on the bottom-right of the highlight is still significantly transparent, while it should be opaque. |
@calixteman , @timvandermeij , @nicolo-ribaudo As soon as you can, could you take a look at it? I think I've fixed the problems. I made a slight change, making the floating bar an “independent” element, moving the floating toolbar to AnnotationEditorLayer to keep it opaque, and maintaining mix-blend-mode to avoid overlaps and making the selection lighter by adding 0.25 as the color opacity. I also added screenshots of how it looks on Brave and Firefox. |















Description
This PR addresses visual artifacts where text selection boxes would overlap incorrectly.
By applying group opacity and using
mix-blend-mode, we ensure that the selection highlight renders correctly on Chrome and other browsers.Issue fixed
Fixes #17561
Type of changes
Checklist