Something is wrong with document anchor links #353

Closed
ibc opened this Issue Jan 16, 2016 · 7 comments

Projects

None yet

3 participants

@ibc
Contributor
ibc commented Jan 16, 2016

I'm reading "The MIME media type. Valid types are listed in [IANA-RTP-2]. The name must always be provided.". The link [IANA-RTP-2] points to is:

http://ortc.org/wp-content/uploads/2015/11/ortc.html#bib-IANA-RTP-2

By opening such a link I do not see the "IANA-RTP-2" bullet at all but, depending on the browser, "[RFC4585]" or any other.

This happens in most of the internal anchor links and it makes very hard to jump within the spec.

@robin-raymond
Contributor

It's not just that link. It's also the TOC. Seems that anchors don't always work very well in general. Sometimes they work. Sometimes they don't.

@ibc
Contributor
ibc commented Jan 16, 2016

It seems that the generated web does not use "common" anchor links. For example, there is not a section with id="bib-IANA-RTP-2", so when we click on http://ortc.org/wp-content/uploads/2015/11/ortc.html#bib-IANA-RTP-2 some script intercepts it, gets the desired section from the URL fragment component, and scrolls down the web...

But that is obviously buggy and does not work in most of the cases. I expect something wrong in the respec-w3c-common script, but who knows.

@aboba aboba added the 1.1 label Jan 17, 2016
@aboba
Contributor
aboba commented Jan 17, 2016

Changed [TSVWG-RTCWEB-QOS] reference to normative from informative and now links seem to work:
http://internaut.com:8080/~baboba/ortc/github/ortc.html

@ibc
Contributor
ibc commented Jan 17, 2016

Wow!

@aboba aboba added a commit that referenced this issue Jan 17, 2016
@aboba aboba Something is wrong with document anchor links
A small change that seems to fix a bunch of the reference links.  Not sure why, exactly. 

Related to Issue #353
7c4c109
@aboba
Contributor
aboba commented Jan 17, 2016

That this minor an issue would break respec is beyond comprehension. So I'm hesitant to say that the problem is really fixed now. Please weigh in if you believe that things are fully working... if not, please let me know which links remain broken and I will investigate further.

@aboba aboba added a commit that referenced this issue Jan 17, 2016
@aboba aboba Fix voiceActivityFlag typo and added change log
In line 3901, "v" was used instead of "voiceActivityFlag".  Also added a change log reference to Issue #353
541c283
@aboba aboba added the PR exists label Jan 18, 2016
@aboba
Contributor
aboba commented Jan 18, 2016

One outstanding broken link left: to WebRTC 1.0. Right now this is informative, would need to promote it to normative for the link to function correctly.

@aboba aboba added a commit that referenced this issue Feb 3, 2016
@aboba aboba WebRTC 1.0 as normative reference
Portions of the ORTC API are derived from WebRTC 1.0, including the sections on Data Channel, DTMF and Identity.  Therefore it is proposed that the WebRTC 1.0 API be made a normative reference.  This change also addresses Issue #353
46e83ce
@aboba
Contributor
aboba commented Feb 3, 2016

Once PR 371 is merged, I believe we can close this issue.

@aboba aboba closed this Feb 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment