Malvoz

Results 195 comments of Malvoz

Could this include checking for compliance with the prerequisites for cookies with [`__Secure-`](https://tools.ietf.org/html/draft-ietf-httpbis-cookie-prefixes#section-3.1) and [`__Host-`](https://tools.ietf.org/html/draft-ietf-httpbis-cookie-prefixes#section-3.2) prefixes perhaps?

Also check that CSP applies to appropriate MIME-types, see https://github.com/w3c/webappsec/issues/520. For reference, H5BP [settled on](https://github.com/h5bp/server-configs-apache/issues/187#issuecomment-515510343) matching against `text\/(html|javascript)|application\/pdf|xml` in their apache configs.

FWIW, I believe with [Scroll-linked Animations](https://drafts.csswg.org/scroll-animations-1/ ) there's no need for a `` element.

``/`` isn't supported in IE/Edge; https://caniuse.com/#feat=details

One thing to note is that [`no-cache`](https://tools.ietf.org/html/rfc7234#section-5.2.2.2) does not mean "do not cache" - that would be [`no-store`](https://tools.ietf.org/html/rfc7234#section-5.2.2.3), rather it means "do not serve this content from the cache without...

Please label this: https://github.com/Leaflet/Leaflet.markercluster/labels/accessibility

I tested in Narrator, attaching screen and audio capture (you may have to unmute the videos): ### Before (no contextual information for marker clusters): ### After (with contextual information `aria-label="markers"`):

@ykzeng do you think you can merge this? If there's a need for more reviews, maybe @Falke-design can take a look?

@Falke-Design I wonder if there are any cases where screen readers would then announce "<childCount> markers <childCount>". That shouldn't be the case, and would probably be a screen reader bug....