Skip to content

[ReaderHighlight] rewrite some UI strings#13145

Merged
Frenzie merged 1 commit into
koreader:masterfrom
Commodore64user:new-strings
Jan 26, 2025
Merged

[ReaderHighlight] rewrite some UI strings#13145
Frenzie merged 1 commit into
koreader:masterfrom
Commodore64user:new-strings

Conversation

@Commodore64user

@Commodore64user Commodore64user commented Jan 26, 2025

Copy link
Copy Markdown
Member

what's new

Just a few improvements to UI text, most of them to my very own chagrin.


This change is Reviewable

table.insert(menu_items.long_press.sub_item_table, {
text_func = function()
return T(_("Rate of movement in content selection: %1"), G_reader_settings:readSetting("highlight_non_touch_factor") or 4)
return T(_("Crosshairs speed for text selection: %1"), G_reader_settings:readSetting("highlight_non_touch_factor") or 4)

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.

Crosshair speed (applies everywhere)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I wasn't completely sure if it was supposed to be singular or plural and according to OED is the latter

1.a. In plural. A pair of fine wires crossing at right angles at the focus of an optical instrument or gunsight for use in positioning, aiming, or measuring; a representation of these wires, usually within a circle, typically on a monitor, screen, etc.

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.

Both are in use. Looking in COCA many of the plural occurrences are somewhat ambiguous (i.e., "in the crosshairs" rather than "in my crosshairs" — the latter also occurs plenty of course). If you're attached to it we can keep it.
image

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

well even AI agrees "It’s crosshairs—always plural when referring to the visual alignment tool, like in scopes or selection tools." ;)

@Frenzie Frenzie Jan 26, 2025

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.

AI also says this:

Many professional fields, including military, hunting, and competitive shooting, often use the singular form "crosshair" in their technical documentation and training materials.

NB It may be total nonsense. I wouldn't trust it one way or the other. I get the impression those manuals are more likely to say reticle. It is true, however, that this random manual speaks of a crosshair: https://www.trijicon.com/uploads/page_content/PML1000_Rev0_4x32_3.5x35_Crosshair_ACOG_Manual_.pdf

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Lol that same manual randomly calls them crosshairs 3 times. They should just stick one version of the word.

@Frenzie Frenzie merged commit 0115522 into koreader:master Jan 26, 2025
@Frenzie Frenzie added this to the 2025.01 milestone Jan 26, 2025
@Matthaiks

Copy link
Copy Markdown

"Interval for crosshairs speed increase: 1 second", "Interval for speed increase: %1 seconds"

Missing "crosshairs" in the latter form?

@Commodore64user Commodore64user deleted the new-strings branch January 26, 2025 20:42
@Commodore64user

Commodore64user commented Jan 26, 2025

Copy link
Copy Markdown
Member Author

"Interval for crosshairs speed increase: 1 second", "Interval for speed increase: %1 seconds"

Missing "crosshairs" in the latter form?

Argg, b*llocks! I’ll add it to the other PR, good catch, thanks @Matthaiks

Frenzie added a commit that referenced this pull request Jan 27, 2025
Frenzie added a commit that referenced this pull request Jan 27, 2025
Commodore64user added a commit to Commodore64user/KOReader_fork that referenced this pull request Jan 27, 2025
Commodore64user pushed a commit to Commodore64user/KOReader_fork that referenced this pull request Jan 27, 2025
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
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.

3 participants