Re: [blink-dev] Intent to Remove: CSS property -webkit-highlight

2023-07-19 Thread slightlyoff via Chromestatus
LGTM3 -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chro

Re: [blink-dev] Intent to Remove: CSS property -webkit-highlight

2023-07-06 Thread Rick Byers
On Sun, Jul 2, 2023 at 4:33 PM Yoav Weiss wrote: > LGTM3 > > On Fri, Jun 30, 2023, 21:47 Stephen Chenney wrote: > >> Thanks Mike for looking up github usage. It does indeed do nothing and >> the github refs are all -webkit-highlight: none or initial, which I have to >> assume was also none. We c

Re: [blink-dev] Intent to Remove: CSS property -webkit-highlight

2023-07-02 Thread Yoav Weiss
LGTM3 On Fri, Jun 30, 2023, 21:47 Stephen Chenney wrote: > Thanks Mike for looking up github usage. It does indeed do nothing and the > github refs are all -webkit-highlight: none or initial, which I have to > assume was also none. We checked many of the http-archive links and they > were all th

Re: [blink-dev] Intent to Remove: CSS property -webkit-highlight

2023-06-30 Thread Stephen Chenney
Thanks Mike for looking up github usage. It does indeed do nothing and the github refs are all -webkit-highlight: none or initial, which I have to assume was also none. We checked many of the http-archive links and they were all the one a11y library usage, all value none. Cheers, Stephen. On Fri,

Re: [blink-dev] Intent to Remove: CSS property -webkit-highlight

2023-06-30 Thread Mike Taylor
https://github.com/search?type=code&q=-webkit-highlight+language%3ACSS&l=CSS shows this has been used elsewhere (but I guess has just been useless in clank?). That said, LGTM2. On 7/1/23 4:33 AM, Rick Byers wrote: Removing a prefixed API with no behavior should be trivial, thanks for the cle

Re: [blink-dev] Intent to Remove: CSS property -webkit-highlight

2023-06-30 Thread Rick Byers
Removing a prefixed API with no behavior should be trivial, thanks for the cleanup Stephen :-) However, the UseCounter is surprisingly high with lots of hits in HA: https://chromestatus.com/metrics/css/timeline/popularity/251. Just confirming that you looked at a sample of those hits and found the

[blink-dev] Intent to Remove: CSS property -webkit-highlight

2023-06-29 Thread Stephen Chenney
Contact emailsschen...@chromium.org, spectran...@igalia.com ExplainerNone SpecificationNone Summary The CSS property -webkit-highlight is intended to highlight text, but was never standardized. It has no visible effect in chromium (it is parsed but never used in rendering content). The property