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

Localization fixes #352

Closed
3 of 4 tasks
moo-man opened this issue Mar 24, 2020 · 22 comments
Closed
3 of 4 tasks

Localization fixes #352

moo-man opened this issue Mar 24, 2020 · 22 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request
Projects

Comments

@moo-man
Copy link
Collaborator

moo-man commented Mar 24, 2020

Untranslatable Strings

  • Characteristic Abbreviations

  • Missing LoreDescription.Shadows in en.json

Bugs

  • /char - career button

  • Items - Size (All specifications as well), Hardy, Small, Mindless, Construct, Armour, Swarm

@moo-man moo-man added bug Something isn't working enhancement New feature or request labels Mar 24, 2020
@moo-man moo-man self-assigned this Mar 24, 2020
@LeRatierBretonnien
Copy link
Contributor

Allow translation of 'yards' in Range (for example)

@LeRatierBretonnien
Copy link
Contributor

LeRatierBretonnien commented Mar 29, 2020

Description of 'Draught Horse' is wrong in trappings.db : this is not a war horse :)
Same for 'Monkey'
Same for 'Riding Horse'
Same for 'Poney'

Description of 'Saddlebags' is wrong

@LeRatierBretonnien
Copy link
Contributor

Enable translation of money : Brass Penny/Gold Crown/Silver Shilling

@Moilunator
Copy link

Hi Moo Man, I'm French and I help translate the compendium psychologies and I don't find "Love" in it. Is it an oversight or a choice ?

@moo-man
Copy link
Collaborator Author

moo-man commented Mar 30, 2020

@Moilunator Love was an example custom psychology, I'm not sure if that's detailed in the French translation, I would imagine it is, I can send you the text if not.

@Moilunator
Copy link

Moilunator commented Mar 30, 2020

I've Love in the french rulesbook, but Love is not included in the compendium (Camaraderie, Phobia and Trauma are).

@moo-man
Copy link
Collaborator Author

moo-man commented Mar 30, 2020

@Moilunator Sorry my apologies, I misinterpreted. Yeah, looks like an oversight, I'll add it next update

@LeRatierBretonnien
Copy link
Contributor

image

CSS (table/celll?) overflow with "Augmentations", cf snapshot

@LeRatierBretonnien
Copy link
Contributor

LeRatierBretonnien commented Apr 2, 2020

When clicking on a prayer spell (blessing or another), the "Roll" window is displayed, but the skill linked with it is "Fellowship", instead of "Pray" -> this can lead to wrong rolls results.

@moo-man
Copy link
Collaborator Author

moo-man commented Apr 2, 2020

@sladecraven
Is this a localization issue? If you've changed the skill names it would be expected

@LeRatierBretonnien
Copy link
Contributor

@moo-man
OK, yes, skill names are translated. So how to manage this properly ? On your side or on my side ?

@moo-man
Copy link
Collaborator Author

moo-man commented Apr 2, 2020

@sladecraven that'd be my side, I've gotta add support to recognize localized item names

@LeRatierBretonnien
Copy link
Contributor

Ok, clear !

@LeRatierBretonnien
Copy link
Contributor

Weapon qualities must be handled thru 'translation'. For example :
dice-wfrp4e.js: if (weapon.properties.qualities.includes("Impale")
should be (i guess) :
dice-wfrp4e.js: if (weapon.properties.qualities.includes("WFRP4E.Properties.Impale")

@moo-man moo-man added this to In progress in 1.2 Apr 4, 2020
@Antares117
Copy link

Bonjour je viens de voir que les dégats de la grande Ache ne sont pas bon : + au lieu de +6.
Et encore merci pour le travail que vous effectué

@LeRatierBretonnien
Copy link
Contributor

LeRatierBretonnien commented Apr 11, 2020

When using names with a ' inside, the findItem function is failing, and the associated text is not displayed in the chat.
Example : In french, the translation of "Ear Bash" is "Frappe à l'oreille" -> this outputs an error in line 801 of dice-wfrp4e.js (dev-backend branch) when clicking in the critical name in the chat.
Without the ' (ie "Frappe à l oreille"), it does not raise the eror
->FIXED on FR SIDE - Wrong char. used by translator

@LeRatierBretonnien
Copy link
Contributor

LeRatierBretonnien commented Apr 11, 2020

When searching a skill for a weapon, the search key is not enough robust.
In actor/actor-wfrp4e.js, the line :
weapon.skillToUse = skills.find(x => x.name.toLowerCase().includes(weapon.data.weaponGroup.value.toLowerCase()))
Should be :
weapon.skillToUse = skills.find(x =>x.name.toLowerCase().includes("("+weapon.data.weaponGroup.value.toLowerCase()+")"))
`

This avoid selection of wrong skill when a part of the name is matching. This is the case for example for the word Arc (ie Bow in french) that match the skill Marchandage (ie Haggle in French).

@LeRatierBretonnien
Copy link
Contributor

LeRatierBretonnien commented Apr 11, 2020

Allow translate of distance unit : yd/yds/yards should be keyed in lang/endjson (replaced by meters in other translations).

@LeRatierBretonnien
Copy link
Contributor

/cond command should search in WFRP4E.conditions string, not keys, in order to allow searching in translated names.

@LeRatierBretonnien
Copy link
Contributor

LeRatierBretonnien commented Apr 13, 2020

in actor-sheet.js, in line 361 :
let improvId = weapons.find(w => w.name.toLowerCase() == "improvised weapon");
"improvised weapon" -> should be localized with NAME.Improvised

@LeRatierBretonnien
Copy link
Contributor

In actor-sheet.js, line 382,
let skill = this.actor.items.find(s => s.data.name == game.i18n.localize("NAME.Enurance") && s.type == "skill")
"NAME.Enurance" -> should be NAME.Endurance

@JiDW
Copy link
Contributor

JiDW commented Apr 13, 2020

the utility function "allMoneyItems" is using English labels to look into the compendium for adding GC/SS/BP to new actors.

utility-wfrp4e.js, line 679

@moo-man moo-man moved this from In progress to Done in 1.2 Apr 15, 2020
@moo-man moo-man closed this as completed Apr 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
No open projects
1.2
  
Done
Development

No branches or pull requests

5 participants