Skip to content

Finna 4288 fix content skip link position 2 #3548

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.

Sign up for GitHub

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

Open
JJManner wants to merge 8 commits into
base: dev
Choose a base branch
from
+60 −19
Open

Finna 4288 fix content skip link position 2 #3548

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
ff8030c
layout- result- default core fixed and skiplink-core-heading added
JJManner Aug 25, 2026
150d1d1
composer qa fix
JJManner Aug 25, 2026
ced448b
core files edited
JJManner Aug 26, 2026
b50b1fe
all skip links and core files fixed
JJManner Aug 31, 2026
277af92
multiple files fixed
JJManner Sep 22, 2026
869ebdb
removed unnecessary variables and named id better
JJManner Oct 1, 2026
dada18f
more fixes
JJManner Oct 19, 2026
cc3646d
core and view files fixed
JJManner Oct 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions themes/finna2/templates/RecordDriver/DefaultRecord/core.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@
$binding = $this->driver->tryMethod('getBinding');
$iiifManifests = $this->driver->tryMethod('getIiifManifests');

// Add Finna skip link target to avoid landing into finnaMainTabs
$this->layout()->finnaSkipLinkTarget = 'record-skip-link-target';

$addRenderedUrl = function (array $url) use (&$renderedURLs) {
$renderedURLs[] = [
'url' => $url['url'],
Expand Down Expand Up @@ -85,8 +88,7 @@

<?php $this->metadata()->generateMetatags($this->driver);?>
<div class="media">
<h1 class="record-title visible-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>

<h1 class="record-title visible-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?php if (!empty($this->extraControls)): ?>
<?=$this->extraControls['actionControls'] ?? ''?>
<?=$this->extraControls['availabilityInfo'] ?? ''?>
Expand All @@ -96,7 +98,7 @@
<?=$thumbnail ?>
<?php endif; ?>
<div class="media-body record-information">
<h1 class="record-title hidden-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?=$this->partial('RecordDriver/DefaultRecord/record-title-skiplink.phtml') ?>
<?php foreach ($this->driver->tryMethod('getFullTitlesAltScript', [], []) as $altTitle): ?>
<div class="record-title-alt-script">
<?=$this->escapeHtml($altTitle)?>
Expand Down
4 changes: 4 additions & 0 deletions themes/finna2/templates/RecordDriver/DefaultRecord/record-title-skiplink.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
<!-- START of: finna - RecordDriver/DefaultRecord/record-title-skiplink.phtml -->
<div role="heading" aria-level="1" id="record-skip-link-target" class="visually-hidden" aria-label="<?=$this->escapeHtml($this->driver->getTitle())?>"></div>
<h1 class="record-title hidden-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<!-- END of: finna - RecordDriver/DefaultRecord/record-title-skiplink.phtml -->
7 changes: 5 additions & 2 deletions themes/finna2/templates/RecordDriver/SolrAipa/core.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,16 +43,19 @@
$img = $this->recordImage($this->record($this->driver));
$thumbnail = false;
$thumbnailAlignment = false;

// Add Finna skip link target
$this->layout()->finnaSkipLinkTarget = $finnaSkipLinkTarget = 'record-skip-link-target';
?>

<?php $this->metadata()->generateMetatags($this->driver);?>
<div class="media">
<h1 class="record-title visible-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<h1 class="record-title visible-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?php if ($thumbnail && $thumbnailAlignment == 'left'): ?>
<?=$thumbnail ?>
<?php endif; ?>
<div class="media-body record-information">
<h1 class="record-title hidden-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?=$this->partial('RecordDriver/DefaultRecord/record-title-skiplink.phtml') ?>
<?php if ($results = $this->driver->getAlternativeTitles()): ?>
<div class="recordAltTitles record-alt-titles">
<?=implode('<br>', array_map([$this, 'escapeHtml'], $results))?>
Expand Down
8 changes: 6 additions & 2 deletions themes/finna2/templates/RecordDriver/SolrEad/core.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,10 @@
$img = $this->recordImage($this->record($this->driver));
$thumbnail = false;
$thumbnailAlignment = $this->record($this->driver)->getThumbnailAlignment('result');

// Add Finna skip link target
$this->layout()->finnaSkipLinkTarget = $finnaSkipLinkTarget = 'record-skip-link-target';

ob_start(); ?>
<div class="media-<?=$thumbnailAlignment ?><?=!empty($audioUrls) ? ' audio' : ''?>">
<?=$this->record($this->driver)->renderTemplate('record-image-information.phtml', ['img' => $img, 'sizes' => ['small' => ['w' => 100, 'h' => 100], 'medium' => ['w' => 1200, 'h' => 1200]]]);?>
Expand All @@ -47,12 +51,12 @@

<?php $this->metadata()->generateMetatags($this->driver);?>
<div class="media">
<h1 class="record-title visible-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<h1 class="record-title visible-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?php if ($thumbnail && $thumbnailAlignment == 'left'): ?>
<?=$thumbnail ?>
<?php endif; ?>
<div class="media-body record-information">
<h1 class="record-title hidden-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?=$this->partial('RecordDriver/DefaultRecord/record-title-skiplink.phtml') ?>
<?php if ($results = $this->driver->getAlternativeTitles()): ?>
<div class="recordAltTitles record-alt-titles">
<?=implode('<br>', array_map([$this, 'escapeHtml'], $results))?>
Expand Down
13 changes: 9 additions & 4 deletions themes/finna2/templates/RecordDriver/SolrForward/core.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,10 @@

$thumbnail = false;
$thumbnailAlignment = $this->record($this->driver)->getThumbnailAlignment('result');

// Add Finna skip link target
$this->layout()->finnaSkipLinkTarget = $finnaSkipLinkTarget = 'record-skip-link-target';

ob_start(); ?>
<div class="media-<?=$thumbnailAlignment ?>">

Expand Down Expand Up @@ -65,19 +69,20 @@

<?php $this->metadata()->generateMetatags($this->driver);?>
<div class="media">
<h1 class="record-title visible-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<h1 class="record-title visible-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?php if ($altTitles): ?>
<div class="alt-title-wrapper visible-xs">
<div class="alt-title-wrapper visible-xs" aria-hidden="true">
<?=$altTitlesContent?>
</div>
<?php endif; ?>
<?php if ($thumbnail && $thumbnailAlignment == 'left'): ?>
<?=$thumbnail ?>
<?php endif; ?>
<div class="media-body record-information">
<h1 class="record-title hidden-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?=$this->partial('RecordDriver/DefaultRecord/record-title-skiplink.phtml') ?>
<?php if ($altTitles): ?>
<div class="alt-title-wrapper hidden-xs">
<div class="visually-hidden"><?=$altTitlesContent?></div>
<div class="alt-title-wrapper hidden-xs" aria-hidden="true">
<?=$altTitlesContent?>
</div>
<?php endif; ?>
Expand Down
8 changes: 6 additions & 2 deletions themes/finna2/templates/RecordDriver/SolrLido/core.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@

$thumbnail = false;
$thumbnailAlignment = $this->record($this->driver)->getThumbnailAlignment('result');

// Add Finna skip link target
$this->layout()->finnaSkipLinkTarget = $finnaSkipLinkTarget = 'record-skip-link-target';

ob_start();
?>
<div class="media-<?=$thumbnailAlignment ?><?=!empty($audioUrls) ? ' audio' : ''?>">
Expand Down Expand Up @@ -72,12 +76,12 @@
?>
<?php $this->metadata()->generateMetatags($this->driver);?>
<div class="media">
<h1 class="record-title visible-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<h1 class="record-title visible-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?php if ($thumbnail && $thumbnailAlignment == 'left'): ?>
<?=$thumbnail ?>
<?php endif; ?>
<div class="media-body record-information">
<h1 class="record-title hidden-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?=$this->partial('RecordDriver/DefaultRecord/record-title-skiplink.phtml') ?>
<?php if ($results = $this->driver->getAlternativeTitles()): ?>
<div class="recordAltTitles record-alt-titles">
<?=implode('<br>', array_map([$this, 'escapeHtml'], $results))?>
Expand Down
8 changes: 6 additions & 2 deletions themes/finna2/templates/RecordDriver/SolrLrmi/core.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,10 @@
$thumbnail = false;
$thumbnailAlignment = $this->record($this->driver)->getThumbnailAlignment('result');
$formats = $this->driver->getFormats();

// Add Finna skip link target
$this->layout()->finnaSkipLinkTarget = $finnaSkipLinkTarget = 'record-skip-link-target';

@EreMaijala EreMaijala Oct 21, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Ja täällä edelleen tuo $finnaSkipLinkTarget-muuttuja..


ob_start(); ?>
<div class="media-<?=$thumbnailAlignment?>">
<?= $this->record($this->driver)->renderTemplate('record-image-information.phtml', ['img' => $img, 'imageClick' => 'none']) ?>
Expand All @@ -25,12 +29,12 @@
<?php $thumbnail = ob_get_contents(); ?>
<?php ob_end_clean(); ?>
<div class="media">
<h1 class="record-title visible-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<h1 class="record-title visible-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?php if ($thumbnail && $thumbnailAlignment == 'left'): ?>
<?=$thumbnail ?>
<?php endif; ?>
<div class="media-body record-information">
<h1 class="record-title hidden-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?=$this->partial('RecordDriver/DefaultRecord/record-title-skiplink.phtml') ?>
<?php if ($results = $this->driver->getAlternativeTitles()): ?>
<div class="recordAltTitles record-alt-titles">
<?=implode('<br>', array_map([$this, 'escapeHtml'], $results))?>
Expand Down
8 changes: 6 additions & 2 deletions themes/finna2/templates/RecordDriver/SolrQdc/core.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,10 @@
$img = $this->recordImage($this->record($this->driver));
$thumbnail = false;
$thumbnailAlignment = $this->record($this->driver)->getThumbnailAlignment('result');

// Add Finna skip link target
$this->layout()->finnaSkipLinkTarget = $finnaSkipLinkTarget = 'record-skip-link-target';

ob_start(); ?>
<div class="media-<?=$thumbnailAlignment ?><?=!empty($audioUrls) ? ' audio' : ''?>">

Expand All @@ -69,12 +73,12 @@

<?php $this->metadata()->generateMetatags($this->driver);?>
<div class="media">
<h1 class="record-title visible-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<h1 class="record-title visible-xs" aria-hidden="true"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?php if ($thumbnail && $thumbnailAlignment == 'left'): ?>
<?=$thumbnail ?>
<?php endif; ?>
<div class="media-body record-information">
<h1 class="record-title hidden-xs"><?=$this->escapeHtml($this->driver->getTitle())?></h1>
<?=$this->partial('RecordDriver/DefaultRecord/record-title-skiplink.phtml') ?>
<?php if ($results = $this->driver->getAlternativeTitles()): ?>
<div class="recordAltTitles record-alt-titles">
<?=implode('<br>', array_map([$this, 'escapeHtml'], $results))?>
Expand Down
4 changes: 4 additions & 0 deletions themes/finna2/templates/layout/layout.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -161,6 +161,10 @@
$contentElementId = 'main';
if (!empty($this->layout()->finnaUserAccountHeading)) {
$contentElementId = 'useraccount-content-heading';
} elseif ($isLandingPage) {
$contentElementId = 'main';
} elseif ((!empty($this->layout()->finnaSkipLinkTarget))) {
$contentElementId = $finnaSkipLinkTarget;

@EreMaijala EreMaijala Oct 8, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Mikä muuttuja tuo $finnaSkipLinkTarget on?

} elseif (!empty($this->layout()->finnaMainHeader)) {
$contentElementId = 'content-heading';
} elseif (!empty($this->layout()->finnaMainTabs)) {
Expand Down
6 changes: 5 additions & 1 deletion themes/finna2/templates/search/results.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,9 @@
$searchType = $this->params->getSearchType();
$geoFilter = '';

// Add Finna skip link target to avoid landing into searchTabs
$this->layout()->finnaSkipLinkTarget = 'result-skip-link-target';

// Store search id:
$this->layout()->searchId = $this->results->getSearchId();

Expand Down Expand Up @@ -110,7 +113,8 @@
$recommendations = $this->results->getRecommendations('side');
?>

<h1 tabindex="-1" id="results-heading" class="visually-hidden"><?= $this->transEsc('Search Results'); ?></h1>
<h1 tabindex="-1" id="results-heading" class="visually-hidden" aria-hidden="true"><?= $this->transEsc('Search Results'); ?></h1>
<div id="<?=$this->layout()->finnaSkipLinkTarget ?>" role="heading" aria-level="1" class="visually-hidden" ><?= $this->transEsc('Search Results'); ?></div>
<?php if ($recordTotal > 0): ?>
<div class="mobile-nav-toggle-trigger visible-xs hidden-print"></div>
<div class="mobile-nav-toggle visible-xs hidden-print">
Expand Down
Loading