Skip to content

Commit

Permalink
NBNP-390 Consistent fulltext search label+improved readability score
Browse files Browse the repository at this point in the history
  • Loading branch information
jtmcd75 committed Jun 21, 2024
1 parent afb6ba2 commit eb00bc4
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
2 changes: 1 addition & 1 deletion configuration/views.view.digital_page_lister.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2840,7 +2840,7 @@ display:
exposed: true
expose:
operator_id: search_api_fulltext_op
label: 'Search within digitized content'
label: 'Search within our scanned newspaper content'
description: '[note] Overwritten via newspapers_core_form_alter()'
use_operator: false
operator: search_api_fulltext_op
Expand Down
4 changes: 2 additions & 2 deletions custom/modules/newspapers_core/newspapers_core.module
Original file line number Diff line number Diff line change
Expand Up @@ -273,8 +273,8 @@ function newspapers_core_form_views_exposed_form_alter(&$form, FormStateInterfac
break;

case 'views-exposed-form-digital-page-lister-page-search':
$extended_description = t('Find 1 or more terms within the <strong>fulltext</strong> of our digitized newspaper
pages. ') . '<br><a aria-controls="searchTips" aria-expanded="false" class="btn btn-link btn-sm text-nowrap"
$extended_description = t('Find 1 or more terms within the fulltext of our digitized newspaper
pages') . '<br><a aria-controls="searchTips" aria-expanded="false" class="btn btn-link btn-sm text-nowrap"
data-toggle="collapse" href="#searchTips" role="button">Search tips<span aria-hidden="true"
class="fa-solid fa-info-circle ml-1"></span></a>';

Expand Down
5 changes: 3 additions & 2 deletions custom/modules/newspapers_core/src/Form/HomePageForm.php
Original file line number Diff line number Diff line change
Expand Up @@ -289,9 +289,10 @@ class="fa-solid fa-circle-question ml-1"></span>')), $about_url
];
$form['panel-wrapper']['tab-content']['fulltext']['search']['input_fulltext'] = [
'#type' => 'search',
'#title' => $this->t('Search the fulltext of digitized newspapers <span class="visually-hidden">required</span>'),
'#title' => $this->t('Search within our scanned newspaper content <span class="visually-hidden">required</span>'),
'#description' => '<span class="mr-1">' .
$this->t('Search for 1 or more keywords within the fulltext of digitized newspapers') . '</span>',
$this->t('Find 1 or more terms within the fulltext of our digitized newspaper pages') .
'</span>',
];
$form['panel-wrapper']['tab-content']['fulltext']['search']['actions'] = [
'#type' => 'actions',
Expand Down

0 comments on commit eb00bc4

Please sign in to comment.