Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ampersands in German translation #5754

Closed
funnym0nk3y opened this issue Dec 16, 2019 · 7 comments
Closed

Ampersands in German translation #5754

funnym0nk3y opened this issue Dec 16, 2019 · 7 comments
Labels
bug Confirmed bugs or reports that are very likely to be bugs internationalization i18n ui
Milestone

Comments

@funnym0nk3y
Copy link

funnym0nk3y commented Dec 16, 2019

JabRef version 5.0-beta on Windows 10

Thanks for the new beta build!
The new beta build has a few, uncritical, quirks I want you to inform of.

In the German translation of JabRef are a few cases where the word is preceded by an ampersand (&).
This is the case in the following menues:
Datei -> Importieren -> Importieren in aktuelle &Bibliothek
Datei -> Importieren -> &Importieren in neue Bibliothek
Werkzeuge -> &XMP-Metadaten in PDFs schreiben
Werkzeuge -> &Einträge in extern Anwendung einfügen
Werkzeuge -> &Abkürzung der Zeitschriftentitel aufheben

EDIT:
Hope a quick update of the translation files fixes the issue. See translated file.

@Siedlerchr
Copy link
Member

Siedlerchr commented Dec 16, 2019

Hi, these ampersands are from the old version and indicated the mmemonic key (e..g underline the accelerator with alt).
Seems like they can be replaced with underscores:
https://medium.com/@zoha131/handling-keyboard-shortcuts-in-javafx-2972ba950a48

Seems like this could work. Would be nice if you or someone else could test this.
https://docs.oracle.com/javase/9/docs/api/javafx/scene/control/MenuItem.html#mnemonicParsingProperty--

Related: https://stackoverflow.com/a/24500058

@Siedlerchr Siedlerchr added bug Confirmed bugs or reports that are very likely to be bugs internationalization i18n ui good first issue An issue intended for project-newcomers. Varies in difficulty. labels Dec 16, 2019
@funnym0nk3y
Copy link
Author

Unfortunately I have never written Java or anything JVM-based. I can offer to replace the ampersands with underscores, but that's it. Probably won't fix that issue though...

Also please excuse the updates in the localization service, I was a little rash.

@Siedlerchr
Copy link
Member

I will try the next days if the underscore works. Then you can again work on crowdin.

@tobiasdiez tobiasdiez added this to the v5.0 milestone Dec 16, 2019
@tobiasdiez
Copy link
Member

I would prefer to use the setMmemonic method (also discussed in your link). This way we don't clutter the localization files with underscores (what happens if the same string is used in the menu and somewhere else?) and are more flexible it the future. Not sure if we can simply reuse the key shortcut or if a new field is necessary (does this need to be language specific? How do memonics work in say Chinese?)

A lot of questions...maybe it's simpler to just remove the memonic stuff completely.

@tobiasdiez tobiasdiez removed the good first issue An issue intended for project-newcomers. Varies in difficulty. label Dec 16, 2019
@Siedlerchr
Copy link
Member

As far as I know we had the menu properties in a separate file. But I think we got rid of them.

@koppor
Copy link
Member

koppor commented Feb 24, 2020

Do we have accelerator keys if we remove the mnemonic stuff? I really ❤️ keyboard shortcuts. Do we need to re-introduce Menu_xx.properties again?

@calixtus
Copy link
Member

I believe this issue can be closed, as the ampersands have been removed from all the translations.
As accelerator keys just the keyboard shortcuts are now left. I think we could make another issue out of that.

koppor pushed a commit that referenced this issue Dec 1, 2021
3bb4b5f infoclio.ch styles for German: remove non-breaking space delimiters (#5754)
adf28db Create journal-of-health-care-for-the-poor-and-underserved.csl (#5752)
0713a8e Update chinese-gb7714-2005-numeric.csl (#5737)
1cd3754 Update china-national-standard-gb-t-7714-2015-author-date.csl (#5746)
c2536b7 Update china-national-standard-gb-t-7714-2015-numeric.csl (#5745)
f8c1392  Create steel-research-international.csl (#5720)
21fe1f5 Create asian-myrmecology.csl (#5718)
91e9e2b Update harvard-university-of-the-west-of-england.csl (#5734)
dd453d1 fix minor erros in polar-research.csl (#5730)
038a8f5 Remove group around no-date cluster (#5731)
0710b51 remove et-al from bibtex.csl (#5728)
bbd703d Add editorial-director to universite-laval-departement-des-sciences-historiques.csl (#5727)
58ea430 Create german-journal-of-agricultural-economics.csl (#5717)
0654e16 Create scandinavian-journal-of-information-systems.csl (#5716)
ce2d537 Update journal-of-computer-applications-in-archaeology.csl (#5715)
755d3d3 Create human-rights-law-review.csl (#5626)
0feda94 Create journal-of-intercultural-studies.csl (#5709)
ae4756d Update acta-universitatis-agriculturae-sueciae.csl (#5713)
323d9ac Update mohr-siebeck-recht.csl (#5559)
15530a8 Bch corr (#5712)
094a1af Create forschungsjournal-soziale-bewegungen-fjsb.csl (#5699)
cb91566 initialize authors and editors (#5714)
2d5cfff Create cancer-biomarkers.csl (#5703)
5e264d5 Update multidisciplinary-digital-publishing-institute.csl (#5708)
46e961f Create klinische-padiatrie.csl (#5711)
e81e877 Create bulletin-archeologique-des-ecoles-francaises-a-l-etranger.csl (#5704)
0029c5a Create polar-research.csl 🧊 (#5702)
7db1361 Update vancouver-imperial-college-london.csl (#5641)
b953e9f Update iso690-author-date-fr-no-abstract.csl (#5706)
91eda8c Update thieme-german.csl (#5710)
ebe0787 Update harvard-imperial-college-london.csl (#5643)
2d4db76 Fix UNESCO IIEP in text
436cbf4 Create revue-archeologique-de-narbonnaise.csl (#5688)
5150bcf Create journal-of-computer-assisted-tomography.csl (#5690)
dd6f050 Create anti-trafficking-review.csl (#5658)
08e622f Create the-angle-orthodontist.csl (#5685)
c6a1907 journal-of-palm-oil-research.csl fix several errors (#5686)
6cbe29d Create bern-university-of-applied-sciences-school-of-agricultural-for… (#5684)
f590dc1 Update biomed-central.csl (#5701)
1efce81 Update turabian-author-date.csl (#5695)
12dbba5 Create tyndale-bulletin (#5673)
b0746db Create Engineered Regeneration (#5682)
e38b953 wikipedia citation template (#5662)
5e7f731 Create early-music-history.csl (#5679)
86443f3 Create zeitschrift-fur-politik.csl (#5676)
68f1996 Create annals-of-work-exposures-and-health.csl (#5666)
1ba9dc6 Create brazilian-journal-of-psychiatry.csl (#5672)
438f92c fix error for speech in ama styles (#5693)
7a0c2d3 set initialize-with-hyphen to false (#5689)
3bd2765 Update emu-austral-ornithology.csl (#5671)
31492b2 fix various errors in natura-croatica.csl (#5687)
94d6b23 Update iso690-author-date-cs.csl (#5677)
5d017da minor update on the "Haute école de gestion de Genève - ISO 690" style (#5665)
2cad8f6 add ibid/subsequent to comparative-politics.csl (#5669)
de0b116 Create taylor-and-francis-vancouver-national-library-of-medicine.csl (#5650)
ed87f99 Update bulletin-de-correspondance-hellenique.csl (#5663)

git-subtree-dir: buildres/csl/csl-styles
git-subtree-split: 3bb4b5f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bugs or reports that are very likely to be bugs internationalization i18n ui
Projects
None yet
Development

No branches or pull requests

5 participants