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

Combat Techniques page duplicated in character sheet (german language setting only) #616

Closed
Infinitum31415 opened this issue Jun 11, 2020 · 5 comments
Assignees
Labels
Bug A bug

Comments

@Infinitum31415
Copy link

Describe the bug
Wenn deutsch als Sprache eingestellt ist, wird die Seite Kampftechniken im Heldenbogen 2x dargestellt (mit minimalem Unterschied im Rüstungen Feld (siehe attachment).
Mindestens mit englisch als eingesteller Sprache, tritt dieser Fehler nicht auf.

To Reproduce

  1. Stelle deutsch als Sprache ein
  2. Starte Optolith neu
  3. Erstelle einen Charakter und beende Charakter-Erschaffung
  4. Öffne Heldenbogen tab.

Expected behavior
Die Kampftechniken Seite sollte nur 1x im Heldenbogen enthalten sein.

Screenshots
image
Rüstungen-Seite 1:
image
Rüstungen-Seite 2:
image

Environment (please complete the following information):

  • OS: Windows 10
  • Version: 1.3.2

Additional context
test.zip
test_character.pdf

@Infinitum31415 Infinitum31415 added the Bug A bug label Jun 11, 2020
@elyukai
Copy link
Owner

elyukai commented Jun 11, 2020

Das ist beabsichtigt, da man sich so aussuchen kann, welchen Bogen man drucken möchte. Es verhält ich in dem Punkt (noch) wie der Selbstrechnende Heldenbogen, der das auf die gleiche Weise macht. Idealerweise sollte der Bogen ausgeblendet werden, wenn keine Zonenrüstung zum Einsatz kommt; umgekehrt sollte das Blatt mit normaler Rüstung ausgeblendet werden, wenn man Zonenrüstung, aber keine normale Rüstung hat. Dieses Feature wird von #407 getrackt.

Was allerdings ein Bug ist, ist, dass er für englische gar nicht angezeigt wird, denn die gibt es ja mittlerweile schon auf Englisch.

@Rahjenaos
Copy link
Collaborator

Rahjenaos commented Jun 17, 2020

Der englische Bogen wird angezeigt wenn als Fallback Language Deutsch eingestellt ist.
Immerhin dann sogar mit den korrekten englischen Begriffen

edit: der Bogen hängt an "Aventurische Rüstkammer 1" und es sieht so aus als gäbe es das nur auf Deutsch

{isBookEnabled (books)
                       (RA.enabledRuleBooks (HA.rules (hero)))
                       (RA.enableAllRuleBooks (HA.rules (hero)))
                       ("US25208")
          ? (
            <CombatSheetZones
              armorZones={armorZones}
              attributes={attributes}
              combatSpecialAbilities={combatSpecialAbilities}
              combatTechniques={combatTechniques}
              derivedCharacteristics={derivedCharacteristics}
              staticData={staticData}
              meleeWeapons={meleeWeapons}
              rangedWeapons={rangedWeapons}
              shieldsAndParryingWeapons={shieldsAndParryingWeapons}
              conditions={conditions}
              states={states}
              useParchment={useParchment}
              />
          )
          : null}```

@Rahjenaos
Copy link
Collaborator

Visibility of the Hit Zone ArmorTab is also linked to "Aventurische Rüstkammer 1". You cannot select that rule book unless you set German as fallback language but you will have a perfectly translated tab afterwards.
is that by intention?

@elyukai
Copy link
Owner

elyukai commented Jun 18, 2020

No. Well, kind of. The English Armory has been added recently but I forgot to update that tab for that. It's not an elegant solution in general, though. It should be in the info of the books if there should be a Hit Zone Armor tab. But the current solution should be ok for now.

@elyukai elyukai added this to Backlog in v1.5.x via automation Feb 1, 2022
@elyukai elyukai added this to the 1.5.0 milestone Feb 1, 2022
@elyukai elyukai self-assigned this Feb 2, 2022
@elyukai elyukai moved this from Backlog to Active in v1.5.x Feb 3, 2022
@elyukai
Copy link
Owner

elyukai commented Feb 3, 2022

The combat sheet with hit zone armor is shown on demand and #1328 has been fixed, so the issue in the code discussed here does not apply anymore.

@elyukai elyukai closed this as completed Feb 3, 2022
v1.5.x automation moved this from Active to Done, not released Feb 3, 2022
@elyukai elyukai removed this from Done, not released in v1.5.x Feb 3, 2022
@elyukai elyukai removed this from the 1.5.0 milestone Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A bug
Projects
None yet
Development

No branches or pull requests

3 participants