20703 2013-01-18 03:33:32 +0000 Make this xref (the-id-attribute) work. 2013-03-15 23:06:50 +0000 1 1 1 Unclassified WHATWG HTML unspecified Other other RESOLVED FIXED http://www.whatwg.org/specs/web-apps/current-work/#the-id-attribute P3 normal Unsorted 1 contributor ian ian kennyluck mike contributor oldest_to_newest 81555 0 contributor 2013-01-18 03:33:32 +0000 Specification: http://www.whatwg.org/specs/web-apps/current-work/multipage/elements.html#the-id-attribute Multipage: http://www.whatwg.org/C#the-id-attribute Complete: http://www.whatwg.org/c#the-id-attribute Comment: Make this xref (the-id-attribute) work. Posted from: 221.223.50.90 by kennyluck@csail.mit.edu User agent: Mozilla/5.0 (compatible; MSIE 10.0; Windows NT 6.1; WOW64; Trident/6.0) 81556 1 1315 kennyluck 2013-01-18 03:36:45 +0000 Created attachment 1315 dfn.js tweak I don't claim that this is fully tested, but it might work. Is there a public repository for dfn.js and such? 84373 2 ian 2013-03-13 23:26:40 +0000 By "work", do you just mean "be more aggressive about including an ID in the back-reference panel"? It seems to work for me otherwise... I've added the back-reference part of your suggested patch. I couldn't work out why the other part would be needed (the ToC doesn't include <dfn>s, does it?). There's no repo for this, it's just a file on my site currently. 84403 3 kennyluck 2013-03-14 11:58:20 +0000 (In reply to comment #2) > By "work", do you just mean "be more aggressive about including an ID in the > back-reference panel"? It seems to work for me otherwise... What I am saying is that clicking neither "id" or "the id attribute" pop up the xref list. > I've added the back-reference part of your suggested patch. I couldn't work > out why the other part would be needed (the ToC doesn't include <dfn>s, does > it?). The Toc include links to <dfn>-like and so dfnGetCaption(a-link-in-toc) would be null because such links are not in any <section> (and such back-reference has no value anyway). By the way, check the console and it raises ReferenceError: parentNode is not defined if you click on "id" as in "the id attribute". > There's no repo for this, it's just a file on my site currently. OK. 84448 4 ian 2013-03-14 22:51:59 +0000 The links in the ToC are in the "Full table of contents" section, which seems to work fine in the popup. I fixed my dumb typo that was giving the error, sorry about that. Does it all work as you desire now? 84449 5 kennyluck 2013-03-14 22:56:12 +0000 (In reply to comment #4) > The links in the ToC are in the "Full table of contents" section, which > seems to work fine in the popup. > > I fixed my dumb typo that was giving the error, sorry about that. Does it > all work as you desire now? No. OK, I see what's going on. It works fine in the Complete version, not the Multipage version now. 84497 6 ian 2013-03-15 23:06:50 +0000 Got it. Fixed! 1315 2013-01-18 03:36:45 +0000 2013-01-18 03:36:45 +0000 dfn.js tweak dfn-for-the-id-attribute.patch text/plain 1249 kennyluck ZGlmZiAtLWdpdCBhL2Rmbi5qcyBiL2Rmbi5qcwppbmRleCA3MWM3M2M4Li4xNzA4NjIyIDEwMDY0 NAotLS0gYS9kZm4uanMKKysrIGIvZGZuLmpzCkBAIC02MCwxMSArNjAsMTEgQEAgZnVuY3Rpb24g ZGZuU2hvdyhldmVudCkgewogICAgIGlmIChub2RlKSB7CiAgICAgICB2YXIgcGFuZWwgPSBkb2N1 bWVudC5jcmVhdGVFbGVtZW50KCdkaXYnKTsKICAgICAgIHBhbmVsLmNsYXNzTmFtZSA9ICdkZm5Q YW5lbCc7Ci0gICAgICBpZiAobm9kZS5pZCkgeworICAgICAgaWYgKG5vZGUuaWQgfHwgbm9kZS5w YXJlbnROb2RlLmlkKSB7CiAgICAgICAgIHZhciBwZXJtYWxpbmtQID0gZG9jdW1lbnQuY3JlYXRl RWxlbWVudCgncCcpOwogICAgICAgICB2YXIgcGVybWFsaW5rQSA9IGRvY3VtZW50LmNyZWF0ZUVs ZW1lbnQoJ2EnKTsKLSAgICAgICAgcGVybWFsaW5rQS5ocmVmID0gJyMnICsgbm9kZS5pZDsKLSAg ICAgICAgcGVybWFsaW5rQS50ZXh0Q29udGVudCA9ICcjJyArIG5vZGUuaWQ7CisgICAgICAgIHBl cm1hbGlua0EuaHJlZiA9ICcjJyArIChub2RlLmlkIHx8IG5vZGUucGFyZW50Tm9kZS5pZCk7Cisg ICAgICAgIHBlcm1hbGlua0EudGV4dENvbnRlbnQgPSAnIycgKyAobm9kZS5pZCB8fCBub2RlLnBh cmVudE5vZGUuaWQpOwogICAgICAgICBwZXJtYWxpbmtQLmFwcGVuZENoaWxkKHBlcm1hbGlua0Ep OwogICAgICAgICBwYW5lbC5hcHBlbmRDaGlsZChwZXJtYWxpbmtQKTsKICAgICAgIH0KQEAgLTg2 LDYgKzg2LDEwIEBAIGZ1bmN0aW9uIGRmblNob3coZXZlbnQpIHsKICAgICAgICAgZm9yICh2YXIg aSA9IDA7IGkgPCBzb3VyY2VMaW5rcy5sZW5ndGg7IGkgKz0gMSkgewogICAgICAgICAgIHZhciBs aW5rID0gc291cmNlTGlua3NbaV07CiAgICAgICAgICAgdmFyIGhlYWRlciA9IGRmbkdldENhcHRp b24obGluayk7CisgICAgICAgICAgLy8gVGhlIGNhcHRpb24gY291bGQgYmUgbnVsbCBmb3IgYSBs aW5rIGluIHRoZSB0b2MuIEl0IGhhcHBlbnMKKyAgICAgICAgICAvLyB3aGVuLCBzYXksIHRoZSBp ZCBpcyAidGhlLWlkLWF0dHJpYnV0ZSIuIElnbm9yZSBpdCBvZiBjb3Vyc2UuCisgICAgICAgICAg aWYgKCFoZWFkZXIpCisgICAgICAgICAgICBjb250aW51ZTsKICAgICAgICAgICB2YXIgYSA9IGRv Y3VtZW50LmNyZWF0ZUVsZW1lbnQoJ2EnKTsKICAgICAgICAgICBpZiAoIWxpbmsuaWQpCiAgICAg ICAgICAgICBsaW5rLmlkID0gJ2RmblJldHVybkxpbmstJyArIGRmblVuaXF1ZUlkKys7Cg==