Update HTML Sanitizer API version info for Fx147#28777
Update HTML Sanitizer API version info for Fx147#28777
Conversation
|
Tip: Review these changes grouped by change (recommended for most PRs), or grouped by feature (for large PRs). |
|
I am opposed to this. At this stage in the release cycle BCD might set the feature as "preview" to capture that this is in nightly only. However if you put as 148 the toolchain automatically shows data as preview until the release goes live. What we don't do once something is in preview is keep all the old flag and version information. |
|
@hamishwillee in that case, we go with Jake's update in #28774? I had thought we might want to keep the preference info. The version previous to his change was "147", not "Nightly", which is what was causing the confusion. In 147 stable, it is still behind the flag. |
|
Yes, there was nothing wrong with Jake's PR from a BCD perspective - #28774 As noted, you could equally well modify from "148" to "preview". That would also be correct. But you wouldn't bother so close to v148 becoming a release. PS I understand why you want to keep the preference info - it just isn't a BCD "way of working" once things reach preview. |
|
Since we're still more than a month away from Fx148, to me it was unsettling that with the version set as "148", the compat table suggests that the feature is coming in a future release, without any indication that it is already available in Nightly. Or behind a flag in stable. I guess I missed reading the legend more closely, which does say "In development. Supported in a pre-release version."
I'll defer to the judgement from you both, thank you! Closing this now. |
Summary
Context
As discussed with @jakearchibald, this is an update to #28774.
Background
Some notes to capture what's happened in the last few months:
Changes in this PR
Related docs
Channel-wise details are documented on the Experimental features page.
147 makes it available by default in Beta and Developer Edition; but we don't mention those channels in BCD, only the stable release and Nightly.
/cc @hamishwillee for awareness.