Skip to content

FINNA-4028_fix_facet_truncate_accessibility - #3536

Open
JJManner wants to merge 6 commits into
NatLibFi:devfrom
JJManner:FINNA-4028_fix_facet_truncate_accessibility
Open

JJManner wants to merge 6 commits into
NatLibFi:devfrom
JJManner:FINNA-4028_fix_facet_truncate_accessibility

Conversation

@JJManner

Copy link
Copy Markdown

This fix will add a new variable - less_options_ellipsis - and change facet's links to use variables - more_options_ellipsis - and - less_options_ellipsis. Finnish official accessibility authority Traficom requires clarifications to the Finnish terms "Lisää" and "Vähemmän" because they do not describe clearly enough what the terms are used for. Furthermore, term "Lisää" has two meanings in Finnish: to add or more, which makes it even more ambiguous.

@JJManner
JJManner requested a review from EreMaijala August 13, 2026 12:17

@EreMaijala EreMaijala left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

  • Finnassa ei muuteta perus-VuFindin tiedostoja.
  • JS-käännöksissä less_ellipsis ja more_ellipsis pitää säilyttää sellaisenaan ja tarvittaessa lisätä uusia. Muuten vaikuttaa myös muihin kohtiin, joissa käytetään VuFindin truncatea.

@JJManner
JJManner requested a review from EreMaijala September 1, 2026 12:21

@EreMaijala EreMaijala left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sama kommentti pätee edelleen. bootstrap5 tulee perus-VuFindista, eikä JS-käännöstä more_ellipsis voi vaihtaa noin vain.

@JJManner
JJManner requested a review from EreMaijala October 2, 2026 15:11
@JJManner

JJManner commented Oct 2, 2026 •

Copy link
Copy Markdown
Author

Nyt täytyy sanoa, että en oikein tiedä miten ihmeessä tuon voisi korjata muuten kuin kajoamalla johonkin vufind:in omiin käännöksiin. Kävin nyt muuttamassa käännöksen noihin vufind:in - fi.ini - tiedostoon ja vastaaviin muihin, koska millään muulla ei tuo käännös tuntunut muuttuvan millään. Kokeilin vaihtaa sitä myös - themes/finna2/templates/layout/finna-js-translations.phtml - tiedotoon, mutta ilmeisesti vufind aina yliajaa tuon jotenkin.

Jos osaat sanoa paremman tavan saada tuo käännös aikaiseksi niin kaikki vinkit ovat tervetulleita.

@EreMaijala EreMaijala left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Truncatelle voi välittää käytettävät labelit data-attribuutissa data-truncate (ks. esim. themes/bootstrap5/templates/Recommend/TopFacets.phtml).

@JJManner
JJManner requested a review from EreMaijala October 21, 2026 07:31
@JJManner

Copy link
Copy Markdown
Author

Yritin lisätä niitä poikkeuksia antamasi esimerkin - themes/bootstrap5/templates/Recommend/TopFacets.phtml) -mukaisesti, mutta näyttää siltä että bootsrap yliajaa ne aina jotenkin. Muokkasin nyt vain paikallisia kielitiedostoja ja - finna-js-translations.phtml - tiedostoa.

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