Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consider falling back on setAttribute when HTMLMetaElement.prototype.media doesn't exist #33

Open
wisniewskit opened this issue Sep 6, 2022 · 2 comments

Comments

@wisniewskit
Copy link

Hello from the web compatiblity team at Mozilla, and thank you for your hard work! While diagnosing this webcompat.com report, I noticed that in issue #30, code was added to check meta elements' media property, which Firefox does not yet support.

I was wondering if you would be willing to adjust the code to use getAttribute to fetch the media value when HTMLMetaElement.prototype.media is not yet supported? Otherwise as meta-tags are rewritten, they will end up losing their media attribute, and theme engines relying on it will not function as well (as reported in the webcompat issue).

@jsakamoto
Copy link
Owner

@wisniewskit Thank you for letting me know!

This is my fault for the lack of E2E testing on Firefox.

I published the fixed version of this library.

They are still "preview" releases because I would like to do more testing on Firefox, but all usual test cases on the .NET 6.0 platform have been passed. I'll appreciate it if you try them out.

Again, thank you for your report!

@wisniewskit
Copy link
Author

wisniewskit commented Sep 8, 2022

Thank you so much! Funny enough, because we ran across this, Firefox should support the media property as of version 106 as well, but this will help the ESR versions of Firefox, and might also help other older browser versions.

I don't have a dev environment set up right now to test the package, but I've pinged @thepra here and on the webcompat.com issue, just in case they are the devs of Collanon and want to test it out.

If there's anything we can do to help make E2E testing easier (that we aren't already doing), please don't hesitate to join our chat app or ping one of us on GitHub/Bugzilla to start a discussion!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants