kiwix-desktop icon indicating copy to clipboard operation
kiwix-desktop copied to clipboard

The usage of the "Filter language" message is unclear

Open amire80 opened this issue 1 year ago • 6 comments

This is a question about the message "language-searcher-placeholder": "Filter language"

I suspect that it's no actually used anywhere.

I tried finding it in the actual app, but no matter where I clicked, I couldn't see it.

It may be used in void KiwixChoiceBox::showPlaceholder(), but that method doesn't seem to be called anywhere. However, the last time I wrote any C++ code was in 2011, so I might be wrong.

If it's not used, it should be removed.

If it used somewhere, it would be great to clarify in qqq what does it mean exactly or perhaps change the message itself. Maybe changing it to "Filter by language" could be clearer.

amire80 avatar Jan 23 '24 19:01 amire80

@juuz0 Do the old unused strings have been removed properly from the master in English during your effort to rewrite the library?

kelson42 avatar Jan 28 '24 15:01 kelson42

@juuz0 ?

kelson42 avatar Mar 04 '24 07:03 kelson42

@kelson42 Sorry for the delay! But yes, in my knowledge, they were already removed.

Regarding @amire80 question, this particular language-searcher-placeholder can be found at https://github.com/kiwix/kiwix-desktop/blob/693b7b1ea569c80f497db508b880db5de959e0fc/src/kiwixchoicebox.cpp#L215 - as you can see, that full string depends on the type (language, category, etc.)

juuz0 avatar Mar 15 '24 14:03 juuz0

So what left is to add description in qqq to help translator.

mgautierfr avatar Mar 21 '24 08:03 mgautierfr

Not familiar with the qqq.json syntaxes. I am wondering if simply adding context for "searcher-placeholder" is enough for the translations. The prefix just needs a direct translation I believe.

If not I can just add context for all three of the filters.

ShaopengLin avatar May 14 '24 20:05 ShaopengLin

There's no special syntax. Just write the context documentation in the qqq file with the same key as in the en file.

Or you can simply write it as a comment here and I'll add it.

amire80 avatar May 14 '24 21:05 amire80