Bug 196843 – Implement IDREF element reflection

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[James Craig]

## IDREF element reflection

- IDREF element reflection pull request approved by WhatWG, but is awaiting implementations to merge: https://github.com/whatwg/html/pull/3917

Not specific to Accessibility, but there is also an ARIA follow-up.
- The ARIA pull request has been verbally approved; Alice is looking into WPT tests
- pull request: https://github.com/w3c/aria/pull/920
- tests for previous IDL change (generated): https://github.com/web-platform-tests/wpt/commit/4caae418cffa4f803f2fe283a4d3fd7cfaae7461

[Radar WebKit Bug Importer]

<rdar://problem/49836210>

[Chris Dumez]

Manuel, what work remains to be done here?

[Manuel Rego Casasnovas]

(In reply to Chris Dumez from comment #2)
> Manuel, what work remains to be done here?

The implementation work should be ready, unless someone changes on the spec side.
The only remaining bit would be enabling AriaReflectionForElementReferencesEnabled flag by default, but I guess that would need to wait for some spec work.

From the spec side, these properties need to be re-added to ARIA spec.
They were there for a while, but got removed until the HTML spec gets updated:
https://github.com/w3c/aria/pull/1260

The HTML PR is: https://github.com/whatwg/html/pull/3917
I've posted a comment today with the updates to see if it can get merged and unblock the situation.

[Manuel Rego Casasnovas]

(In reply to Manuel Rego Casasnovas from comment #3)
> (In reply to Chris Dumez from comment #2)
> > Manuel, what work remains to be done here?
> 
> The implementation work should be ready, unless something changes on the spec
> side.

It looks like there's a missing bit from the spec that we are not implementing yet. I've reported a separated bug about that, see bug #240563.

[Manuel Rego Casasnovas]

(In reply to Chris Dumez from comment #2)
> Manuel, what work remains to be done here?

An update on the status.

WebKit implementation is feature complete and passes all the tests.

The HTML PR has been merged: https://github.com/whatwg/html/pull/7934

I've sent a PR for ARIA to add these attributes back to the IDL: https://github.com/w3c/aria/pull/1755

So I guess it's up to you to decide when you want to enable this feature by default. I can prepare the patch for that, if you ask me to do it.

[Chris Dumez]

(In reply to Manuel Rego Casasnovas from comment #5)
> (In reply to Chris Dumez from comment #2)
> > Manuel, what work remains to be done here?
> 
> An update on the status.
> 
> WebKit implementation is feature complete and passes all the tests.
> 
> The HTML PR has been merged: https://github.com/whatwg/html/pull/7934
> 
> I've sent a PR for ARIA to add these attributes back to the IDL:
> https://github.com/w3c/aria/pull/1755
> 
> So I guess it's up to you to decide when you want to enable this feature by
> default. I can prepare the patch for that, if you ask me to do it.

I have no opposition to enabling by default.

[James Craig]

> I have no opposition to enabling by default.

Same here. Thanks Rego.

[Manuel Rego Casasnovas]

Pull request: https://github.com/WebKit/WebKit/pull/2151

[EWS]

Committed 252590@main (dcb5be262b06): <https://commits.webkit.org/252590@main>

Reviewed commits have been landed. Closing PR #2151 and removing active labels.

[WebKit Commit Bot]

Re-opened since this is blocked by bug 245429