search

DCLP / dclpxsltbox

Sandbox for development, testing, and review of XSLT for DCLP
http://dclp.github.io/dclpxsltbox/
1 stars 5 forks source link

click through biblio number in principalEdition citation display #133

Open paregorios opened 9 years ago

paregorios commented 9 years ago

At present, the standard biblio-collection code that we're hijacking to serialize citations for bibliography in the principalEdition div of DCLP texts prints first the bibliographic ID number and then the standard-format citation. @jcowey would like to see, when we produce citations inside HTML pages for DCLP items, this id number be a link to the corresponding bibliographic record in HTML.

So, for example:

In the HTML for TM 108908, we find a reference as follows:

75990. John LUNDON, "PSI inv. 320 verso: Fragment einer Rede?," Zeitschrift fur Papyrologie und Epigraphik, 159 (2007).

We'd like the string 75990 linked to the corresponding HTML (i.e., http://server.domain.address/biblio/75990). If possible, it might also be preferable to move this number to the end of the citation, instead of keeping it at the beginning.

Note that these changes will have to be made in such a way that they only occur when we are processing a DCLP XML file for transform. Transformation of a biblio file for the purposes of displaying bibliography must be left untouched.

Relevant intervention point in the code is probably here.

paregorios commented 9 years ago

@jcowey has suggested via Skype that the number be moved to the end of the citation. @rogerbagnall has observed via email that the number as presently deployed is non-sensical to most users. I wonder whether we should not only move it to the end, but gloss it. Another possibility is to suppress the output of the id number entirely, instead linking the title of the article, book, or chapter to the full biblio reference. What do others think? @rla2118 @Edelweiss @HolgerEssler @andreabernini

jcowey commented 9 years ago

Author and title underlined and being the link to the full biblio record, would be my suggestion.

rogerbagnall commented 9 years ago

I agree with James's suggestions of linking, underlining, and number at the end.

rla2118 commented 9 years ago

I also like the idea of linking to the full biblio record, and would be inclined to suppress the number.

On Sat, May 2, 2015 at 6:41 PM, rogerbagnall notifications@github.com wrote:

I agree with James's suggestions of linking, underlining, and number at the end.

— Reply to this email directly or view it on GitHub https://github.com/DCLP/dclpxsltbox/issues/133#issuecomment-98377239.

HolgerEssler commented 9 years ago

I like the idea of linking to the full biblio reference.

paregorios commented 9 years ago

OK. There seems to be consensus around linking, but less so about the number. No need to resolve at the moment; we will have to add logic to detect the fact we're processing for DCLP display anyway (as opposed to BP), so let's get that in train first.

paregorios commented 7 years ago

I think we ought to try to answer #245 before we dig any further into this one, so I'm leaving it unassigned on the backlog for now.

paregorios commented 7 years ago

245 has not yet been addressed, and we are running out of time to start integration testing. As this seems a minor enhancement, recommend defer.

paregorios commented 7 years ago

In my view, #245 is no longer a blocker for this task.

wsalesky commented 7 years ago

@paregorios Clarification question: Links only on principalEdition? Or on all the citations, ie, partial, reference, previous editions.

paregorios commented 7 years ago

On all citations if possible please

On Thu, Aug 31, 2017 at 10:53 AM Winona Salesky notifications@github.com wrote:

@paregorios https://github.com/paregorios Clarification question: Links only on principalEdition? Or on all the citations, ie, partial, reference, previous editions.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/DCLP/dclpxsltbox/issues/133#issuecomment-326320816, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQEdfpNLLT-XOe5yUa32CEu6xBR41jTks5sdsjhgaJpZM4EM5G3 .

-- -- Tom Elliott, Ph.D. Associate Director for Digital Programs and Senior Research Scholar Institute for the Study of the Ancient World (NYU) http://isaw.nyu.edu/people/staff/tom-elliott

Humanities Commons: @paregorios https://hcommons.org/members/paregorios/ OrcID: 0000-0002-4114-6677 http://orcid.org/0000-0002-4114-6677

wsalesky commented 7 years ago

Branch: https://github.com/DCLP/navigator/tree/issue133 Commit: https://github.com/DCLP/navigator/commit/38d6f66bde6dce48c0ac9d6dcb9fac715e5948fa

Changes: Adds links around titles to take users to bibliographic record. Link follows the form already set up in the htm-teibibl.xsl:

<a href="/biblio/{t:idno[@type='pi']}/">[TITLE]</a>

Notes: This link does not work here: http://litpap.info/dclp/62416 or http://dclp.github.io/dclpxsltbox/output/dclp/63/62416.html. I assume this is a system issue not an XSLT issue as it follows the pattern specified in the original request. (Links for journals were already enabled, following the same pattern, and also do not resolve.)

@paregorios This one is ready for review.