Skip to content

Remove implicit anchors attached to titles/whole bibliography #144

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

Merged

Conversation

cormacrelf
Copy link
Contributor

@cormacrelf cormacrelf commented Sep 1, 2021

I propose removing the second half of 'Appendix IV: Links' in CSL 1.1, which provides for adding an anchor to the title of a bibliography entry if the url fields are not rendered at all, or if there is no title, to the entire bibliography entry.

See my comment here: citation-style-language/schema#395 (comment)

Yeah. I think the "if no url rendered in bib entry, add the anchor to the title` fallback thing goes too far. In a nutshell, I don't think anyone wants titles to be blue/underlined but link styling is fine for all the others, and that is quite tricky to resolve at the CSL processor level. More broadly, departing from a convention where every URL is visible in textual form is a bad idea. Think accidental inclusion of incorrect links that someone writing a paper might not even notice have made it in. Think sci-hub.

Expanding on some of those points

This is what I mean by the link styling:

  1. CSL processors don't control styling beyond <i>italic</i> really, they can only do more with markup
  2. Currently consumers of processor output can safely just render it verbatim and not worry that it looks horribly wrong, because the only things that are links are links. HTML by default has blue and underlined. This is fine right now. Links being blue is actually great for many reasons, including in PDFs esp for digital reading.
  3. Occasionally highlighting entire bibliographic references breaks this, now they need to remove all link styling.
  4. Because of Spec: more precise description behavior delimiter-precedes-last #1, would need new markup (class="unstyled-anchor"...) if you want to keep blue/underline for some links. The spec doesn't really talk about markup yet and this would either become another implementation-defined output thing, or require a bunch of spec work. Worthwhile work, but still.

An afterthought from an implementor's perspective, not super important though:

  1. It's actually a lot of work to implement as written. "If a URL field is not rendered" is a question that isn't fully resolved until after disambiguation. Even if you pretend it's resolved early, anything non-local like this is hard, simply because the piece of code rendering the title can't know if the URL will be rendered until later, so it requires adding anchors late in the processing. All of this becomes really twisted when you do split author-in-text citations with a custom in-text part including a title, rendered separately from the rest of the cite. Doesn't really seem worth it from where I stand.

Two more notes on Appendix IV:

  1. On the wording of Appendix IV in general -- I see no reason why this should apply only to bibliography entries, and make no mention of inline/note citations. You want the overall auto-hyperlinking behaviour in both.

  2. The spec should probably also say that url fields that are not valid URLs should not be given an anchor at all. One of the benefits of link styling is that validity == styling+clickability is great for proofing.

@bdarcus
Copy link
Member

bdarcus commented Sep 1, 2021

As I said, I'm fine with removing this part.

On this, anyone have any suggestions?

  1. I see no reason why this should apply only to bibliography entries, and make no mention of inline/note citations. You want the overall auto-hyperlinking behaviour in both.

@bwiernik
Copy link
Member

bwiernik commented Sep 1, 2021

Happy to not link the title. The link styling issue is a big one.

On the wording of Appendix IV in general -- I see no reason why this should apply only to bibliography entries, and make no mention of inline/note citations. You want the overall auto-hyperlinking behaviour in both.

The spec should probably also say that url fields that are not valid URLs should not be given an anchor at all. One of the benefits of link styling is that validity == styling+clickability is great for proofing.

Fine with both of these.

@bdarcus bdarcus merged commit 00eacb0 into citation-style-language:master Sep 2, 2021
@bdarcus bdarcus mentioned this pull request Jun 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants