Skip to content

AO3-7554 i18n - search results help for tags - #5972

Open
sherin wants to merge 5 commits into
otwcode:masterfrom
sherin:tag-search-i18n-text-results
Open

AO3-7554 i18n - search results help for tags#5972
sherin wants to merge 5 commits into
otwcode:masterfrom
sherin:tag-search-i18n-text-results

Conversation

@sherin

@sherin sherin commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Pull Request Checklist

Issue

https://otwarchive.atlassian.net/browse/AO3-7554

Purpose

Move the two tag search help related files to be i18n compatible. As part of that, I've:

  1. Converted the html files to erb
  2. Added relevant texts to the translation file
  3. Added redirect from the old help link to the new one

Testing Instructions

In the jira ticket.

Credit

Sherin (she)

@zz9pzza

zz9pzza commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Hi can you fill in the form and there are some hound comments.

( Sorry didn't see the work in progress )

@sherin sherin changed the title [AO3-7554] update search results help for tags to be i18n compatible AO3-7554 update search results help for tags to be i18n compatible Aug 10, 2026
@sherin
sherin marked this pull request as ready for review August 11, 2026 22:04
@sherin sherin changed the title AO3-7554 update search results help for tags to be i18n compatible AO3-7554 i18n - search results help for tags Aug 24, 2026

@ASalbenblatt ASalbenblatt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just a few fixes I saw, great work though!!

@ASalbenblatt ASalbenblatt Aug 26, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm just putting this here because it comes up a lot in this file, but translators shouldn't be expected to know what kbd and samp mean. I might instead use something more descriptive like as_entered and result respectively (or whatever you want if you can come up with something clearer or better)

refine: If there are too many tags, try refining your search, rather than paging through the results.
sorting: Very new tags will be at the top of the list. Otherwise the list is sorted alphabetically by type and then name.
tags_search_text:
and:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this would be better named space:

double_quote:
description_html: "%{harry_lockhart_kbd} will find %{harry_lockhart_samp} but not %{harry_potter_gilderoy_lockhart_samp}."
harry_lockhart_kbd: '"Harry Lockhart"'
harry_lockhart_samp: '"Harry Lockhart"'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assuming you agree, I think the quotes on the samp were actually a typo in the original, because there would probably only be a tag Harry Lockhart not "Harry Lockhart", so I you can remove them unless you think they might not be one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants