Anchor links in archives

UBWolf · October 30, 2015, 9:36am

Hello there,

is it possible to add anchor links in the archive? We have some journals with quite a few volumes and people have to scroll a long way down. So it would be nice if there could be a shorter way.

Kind regards
Daniela

ctgraham · October 30, 2015, 12:35pm

If you mean for direct hyperlinking from an external source, id selectors can now be used as anchors. On the archives page, you should find ID attributes on each div containing an issue, named “issue-1234”.

github.com

pkp/ojs/blob/ojs-stable-2_4_7/templates/issue/archive.tpl#L31


		{else}
			</div>
			<br />
			<div class="separator" style="clear:left;"></div>
		{/if}
		<div style="float: left; width: 100%;">
		<h3>{$issue->getYear()|escape}</h3>
		{assign var=lastYear value=$issue->getYear()}
	{/if}

	<div id="issue-{$issue->getId()}" style="clear:left;">
	{if $issue->getFileName($locale)}
		{assign var="coverLocale" value="$locale"}
	{else}
		{assign var="coverLocale" value="$primaryLocale"}
	{/if}
	{if $issue->getFileName($coverLocale) && $issue->getShowCoverPage($coverLocale) && !$issue->getHideCoverPageArchives($coverLocale)}
		<div class="issueCoverImage"><a href="{url op="view" path=$issue->getBestIssueId($currentJournal)}"><img src="{$coverPagePath|escape}{$issue->getFileName($coverLocale)|escape}"{if $issue->getCoverPageAltText($coverLocale) != ''} alt="{$issue->getCoverPageAltText($coverLocale)|escape}"{else} alt="{translate key="issue.coverPage.altText"}"{/if}/></a>
		</div>
		<h4><a href="{url op="view" path=$issue->getBestIssueId($currentJournal)}">{$issue->getIssueIdentification()|escape}</a></h4>
		<div class="issueCoverDescription">{$issue->getLocalizedCoverPageDescription()|strip_unsafe_html|nl2br}</div>

Use this ID in your URL the same as the anchor names used to be used:
http:// mysite.tld / ojs / index.php / journalName / issue / archive #issue123

In general, I’d love to see IDs applied like liberally this across the product. If the same were done for <h3>s on this page, we could make a jumplist (which may have been your goal). The sections and articles in the Issue TOC could also benefit from this.

UBWolf · October 30, 2015, 12:56pm

Thank you! But my goal was indeed a jumplist for every year or even for five year intervals. But I assume it’s pretty difficult to do if not impossible at this moment. But maybe there is a way for future OJS versions?

ctgraham · October 30, 2015, 1:33pm

I don’t think the change would be particularly difficult, nor trivial either. The biggest question would be when to display the jumplist. Perhaps if the number years * average issues per year exceeded some value?

You would need to iterate through the published issues in IssueHandler::archive()

github.com

pkp/ojs/blob/ojs-stable-2_4_7/pages/issue/IssueHandler.inc.php#L117


	 * @param $request PKPRequest
	 */
	function archive($args, $request) {
		$this->validate($request);
		$this->setupTemplate();

		$journal =& $request->getJournal();
		$issueDao =& DAORegistry::getDAO('IssueDAO');
		$rangeInfo = $this->getRangeInfo('issues');

		$publishedIssuesIterator = $issueDao->getPublishedIssues($journal->getId(), $rangeInfo);

		import('classes.file.PublicFileManager');
		$publicFileManager = new PublicFileManager();
		$coverPagePath = $request->getBaseUrl() . '/';
		$coverPagePath .= $publicFileManager->getJournalFilesPath($journal->getId()) . '/';

		$templateMgr =& TemplateManager::getManager();
		$templateMgr->assign('coverPagePath', $coverPagePath);
		$templateMgr->assign('locale', AppLocale::getLocale());
		$templateMgr->assign('primaryLocale', $journal->getPrimaryLocale());

Assign the years to be created as a jumplist to an array, and pass the array as a Smarty variable.

Unfortunately, I don’t think you can reuse / reset the existing iterator to pass through the iteration twice, so I think it requires two iterator calls.

UBWolf · November 2, 2015, 8:15am

Thank you I’ll show this my colleagues, maybe they include this.

Kind regards
Daniela