Merge "Use the search title if configured"

This commit is contained in:
jenkins-bot 2021-08-13 18:11:07 +00:00 committed by Gerrit Code Review
commit 040cbb7dc4
2 changed files with 13 additions and 2 deletions

View file

@ -10,6 +10,7 @@
:title="searchTitle"
:placeholder="searchPlaceholder"
:aria-label="searchPlaceholder"
:search-page-title="searchPageTitle"
:initial-input-value="searchQuery"
:button-label="$i18n( 'searchbutton' ).text()"
:form-action="action"
@ -74,6 +75,10 @@ module.exports = {
}
},
props: {
searchPageTitle: {
type: String,
default: 'Special:Search'
},
autofocusInput: {
type: Boolean,
default: false

View file

@ -7,9 +7,11 @@ var
/**
* @param {HTMLElement} searchForm
* @param {HTMLInputElement} search
* @param {string|null} searchPageTitle title of page used for searching e.g. Special:Search
* If null then this will default to Special:Search.
* @return {void}
*/
function initApp( searchForm, search ) {
function initApp( searchForm, search, searchPageTitle ) {
// eslint-disable-next-line no-new
new Vue( {
el: searchForm,
@ -24,6 +26,7 @@ function initApp( searchForm, search ) {
autofocusInput: search === document.activeElement,
action: searchForm.getAttribute( 'action' ),
searchAccessKey: search.getAttribute( 'accessKey' ),
searchPageTitle: searchPageTitle,
searchTitle: search.getAttribute( 'title' ),
searchPlaceholder: search.getAttribute( 'placeholder' ),
searchQuery: search.value
@ -42,9 +45,12 @@ function initApp( searchForm, search ) {
function main( document ) {
var
searchForm = /** @type {HTMLElement} */ ( document.querySelector( '#searchform' ) ),
titleInput = /** @type {HTMLInputElement|null} */ (
searchForm.querySelector( 'input[name=title]' )
),
search = /** @type {HTMLInputElement|null} */ ( document.getElementById( 'searchInput' ) );
if ( search && searchForm ) {
initApp( searchForm, search );
initApp( searchForm, search, titleInput && titleInput.value );
}
}
main( document );