Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix link warning for DVBSS bibliographic entry (#935). #936

Merged
merged 2 commits into from Jul 26, 2018

Conversation

skynavga
Copy link
Collaborator

Closes #935.

@skynavga skynavga added this to the CR3 milestone Jul 25, 2018
@skynavga skynavga self-assigned this Jul 25, 2018
@@ -25772,8 +25772,8 @@ European Broadcasting Union (EBU), March 2015.
<titleref href="https://tech.ebu.ch/publications/tech3370">EBU-TT Part 3 Live Subtitling Applications, Version 1.0</titleref>, European Broadcasting Union (EBU), May 2017.
(See <xspecref href="https://tech.ebu.ch/publications/tech3370">https://tech.ebu.ch/publications/tech3370</xspecref>.)</bibl>
<bibl id="DVBSS" key="DVBSS">ETSI EN 300 743,
<titleref href="http://www.etsi.org/deliver/etsi_en/300700_300799/300743/01.05.01_60/en_300743v010501p.pdf">Digital Video Broadcasting (DVB); Subtitling systems, Version 1.5.1</titleref>, ETSI, January 2014.
(See <xspecref href="http://www.etsi.org/deliver/etsi_en/300700_300799/300743/01.05.01_60/en_300743v010501p.pdf">http://www.etsi.org/deliver/etsi_en/300700_300799/300743/01.05.01_60/en_300743v010501p.pdf (PDF)</xspecref>.)</bibl>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did the (PDF) in the text cause a warning?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. The URL had a permanent redirect. But we don't use "(PDF)" anywhere else, so don't need it here anyway.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well I suppose we should be consistent, and there probably should be some kind of consistency here in the W3C style. My preference would be to warn people when they're not going to a web page, but it's not a big deal.

@@ -25772,8 +25772,8 @@ European Broadcasting Union (EBU), March 2015.
<titleref href="https://tech.ebu.ch/publications/tech3370">EBU-TT Part 3 Live Subtitling Applications, Version 1.0</titleref>, European Broadcasting Union (EBU), May 2017.
(See <xspecref href="https://tech.ebu.ch/publications/tech3370">https://tech.ebu.ch/publications/tech3370</xspecref>.)</bibl>
<bibl id="DVBSS" key="DVBSS">ETSI EN 300 743,
<titleref href="http://www.etsi.org/deliver/etsi_en/300700_300799/300743/01.05.01_60/en_300743v010501p.pdf">Digital Video Broadcasting (DVB); Subtitling systems, Version 1.5.1</titleref>, ETSI, January 2014.
(See <xspecref href="http://www.etsi.org/deliver/etsi_en/300700_300799/300743/01.05.01_60/en_300743v010501p.pdf">http://www.etsi.org/deliver/etsi_en/300700_300799/300743/01.05.01_60/en_300743v010501p.pdf (PDF)</xspecref>.)</bibl>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well I suppose we should be consistent, and there probably should be some kind of consistency here in the W3C style. My preference would be to warn people when they're not going to a web page, but it's not a big deal.

@skynavga skynavga merged commit 2d4eab4 into master Jul 26, 2018
@skynavga skynavga removed their assignment Jul 26, 2018
@skynavga
Copy link
Collaborator Author

Merged early per WG resolution to process CR3 CfC.

@skynavga skynavga deleted the issue-0935-links branch October 4, 2018 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants