Skip to content

fix: a11y in search-results page - #1206

Merged
giuliaghisini merged 7 commits into
mainfrom
US78813-accessibility-fix-search-page
Oct 28, 2026
Merged

giuliaghisini merged 7 commits into
mainfrom
US78813-accessibility-fix-search-page

Conversation

@ilariamassa96

Copy link
Copy Markdown

@giuliaghisini giuliaghisini changed the title fix: fixed hierarchy title in search-results page fix: a11y in search-results page Oct 22, 2026
</Row>
<Skiplink tag="div">
<SkiplinkItem href="#search-results-region" tag="a">
<SkiplinkItem href={`#$"search-results"`} tag="a">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

questo non credo che funzioni scritto così.. come risulta il markup ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

anche l'id "search-results-region" andava bene, non era necessario cambiarlo. Tornerei indietro e manterrei quel nome perchè una modifica di questo tipo potrebbe essere breaking per chi ha fatto personalizzazioni nel tema su quel selettore

Comment thread RELEASE.md Resolved
Comment thread locales/de/LC_MESSAGES/volto.po Outdated
Comment on lines +3979 to +3985
msgid "search_live_input_description"
msgstr ""

#: components/ItaliaTheme/Search/Search
# defaultMessage: {start}-{end} di {total} risultati
msgid "search_results_range"
msgstr ""

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

da mettere le traduzioni in questa lingua

Comment on lines +3962 to +3971
#: components/ItaliaTheme/Search/Search
# defaultMessage: I risultati della ricerca verranno aggiornati in base al testo inserito nel campo di ricerca e ai filtri selezionati
msgid "search_live_input_description"
msgstr ""

#: components/ItaliaTheme/Search/Search
# defaultMessage: {start}-{end} di {total} risultati
msgid "search_results_range"
msgstr ""

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

da mettere le traduzioni in questa lingua

Comment on lines +3971 to +3980
#: components/ItaliaTheme/Search/Search
# defaultMessage: I risultati della ricerca verranno aggiornati in base al testo inserito nel campo di ricerca e ai filtri selezionati
msgid "search_live_input_description"
msgstr ""

#: components/ItaliaTheme/Search/Search
# defaultMessage: {start}-{end} di {total} risultati
msgid "search_results_range"
msgstr ""

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

da mettere le traduzioni in questa lingua

Comment on lines +3979 to +3988
#: components/ItaliaTheme/Search/Search
# defaultMessage: I risultati della ricerca verranno aggiornati in base al testo inserito nel campo di ricerca e ai filtri selezionati
msgid "search_live_input_description"
msgstr ""

#: components/ItaliaTheme/Search/Search
# defaultMessage: {start}-{end} di {total} risultati
msgid "search_results_range"
msgstr ""

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

da mettere le traduzioni in questa lingua

@giuliaghisini
giuliaghisini merged commit 44b87d5 into main Oct 28, 2026
6 checks passed
@giuliaghisini
giuliaghisini deleted the US78813-accessibility-fix-search-page branch October 28, 2026 11:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants