Closed Bug 1868428 Opened 4 months ago Closed 4 months ago

Add ShadowRoot::clonable

Categories

(Core :: DOM: Core & HTML, task)

task

Tracking

()

RESOLVED FIXED
122 Branch
Tracking Status
firefox122 --- fixed

People

(Reporter: saschanaz, Assigned: saschanaz)

References

()

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

A corresponding attribute for ShadowRootInit::clonable.

Pushed by krosylight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ade0be35500e
Add ShadowRoot clonable attribute r=avandolder,webidl,emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/43602 for changes under testing/web-platform/tests
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
Upstream PR merged by moz-wptsync-bot

Hey! I’m currently documenting this addition in BCD, MDN, and release notes. As far as I understand, it’s Nightly only and won’t be shipped in Firefox 122 by default. Did I get it right?

Also, I tried to test it in the current Firefox Nightly 123, but I got false in the console:

const host = document.createElement('div');
const shadowRoot = host.attachShadow({mode: 'open', cloneable: true});
shadowRoot.cloneable;

In Safari TP 186 I get true.

(In reply to Vadim Makeev from comment #6)

Hey! I’m currently documenting this addition in BCD, MDN, and release notes. As far as I understand, it’s Nightly only and won’t be shipped in Firefox 122 by default. Did I get it right?

Yes, it's currently Nightly only. Bug 1870052 will enable it on all channels later.

Also, I tried to test it in the current Firefox Nightly 123, but I got false in the console:

const host = document.createElement('div');
const shadowRoot = host.attachShadow({mode: 'open', cloneable: true});
shadowRoot.cloneable;

In Safari TP 186 I get true.

Hi, it's because it's not cloneable but clonable (see there's no e before able). Looks like Safari has some typo? I filed https://bugs.webkit.org/show_bug.cgi?id=267634.

Oh my, English is hard! I’m glad we did it right as clonable per spec. I’ll make sure there are no cloneable in MDN :)

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: