-
Notifications
You must be signed in to change notification settings - Fork 64
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
Remove " The durability option is new in this edition. It is supported in Chrome 82, Edge 82, and Safari 15. " #424
Comments
When was support added in Firefox? For features new in a given edition of the spec, I usually call out the version it was added in the major browsers, and only remove those banners when that edition is final. If we move to "living standard" (#402) then agreed it could go away. But for now I'd prefer to add the min version the option was supported in FF, rather than remove the banner. |
Per https://developer.mozilla.org/en-US/docs/Web/API/IDBTransaction/durability - looks like FF 126? |
Looks like |
Rather unusual to have inline comments about browser support. They get outdated easily, like in this case. A random web dev might think that only certain browsers support the feature and thus not use it. |
Fair. When the spec was updated more frequently this was less of a problem. Relying on Bikeshed's MDN panels is probably the right way to go, although it appears those may not auto-update... the one for databases() still doesn't show FF support it seems? |
See discussion in #424 - we maybe want to remove these entirely, but this minimal update is harmless. I'm not sure why MDN panels aren't updating.
Durability is supported also in other browsers.
The text was updated successfully, but these errors were encountered: