MidgardVTT-Entwicklung
/
foundry-vtt-system-midgard5
7
0
Fork 0
Code Issues 16 Pull Requests Actions Projects 1 Releases 8 Wiki Activity

Charakterbogen anpassen #7 #9

Merged
oskaloq merged 2 commits from oskaloq into develop 2024-01-03 15:57:18 +01:00
Conversation 3 Commits 2 Files Changed 15 +466 -166
oskaloq commented 2023-12-31 11:03:47 +01:00
Member
Copy Link
  • Use of flexboxes in two columns for better display
  • Introduction of item containers for the different storage of items
- Use of flexboxes in two columns for better display - Introduction of item containers for the different storage of items
oskaloq added 1 commit 2023-12-31 11:03:48 +01:00
a9d7bbb5b2 Revision of the gear tab: 
- Use of flexboxes in two columns for better display
- Introduction of item containers for the different storage of items
oskaloq changed title from Revision of the gear tab to Charakterbogen anpassen #7 2023-12-31 11:09:28 +01:00
Byroks reviewed 2024-01-03 10:03:56 +01:00
Byroks left a comment
Owner
Copy Link

Wir sollten uns überlegen ob wir items in einem Container noch unter "Gegenstand" anzeigen wollen oder nicht.

Auch ob Container Würfe und Mods enthalten dürfen sollten wir abklären. Falls ja, fehlt in M5Character.ts Zeile 153 bei dem Filter noch eine abfrage nach item.type === "container"

Wir sollten uns überlegen ob wir items in einem Container noch unter "Gegenstand" anzeigen wollen oder nicht. Auch ob Container Würfe und Mods enthalten dürfen sollten wir abklären. Falls ja, fehlt in M5Character.ts Zeile 153 bei dem Filter noch eine abfrage nach item.type === "container"
source/module/actors/M5Character.ts
@ -171,6 +173,7 @@ export class M5Character extends Actor {
if (item.system.magic) {
label += "*";
}
let icon = item.img;
Byroks commented 2024-01-03 09:33:51 +01:00
Owner
Copy Link

Wenn wir icons schon mit reinnehmen in das actor sheet, dann sollten wir die in der tabelle auch anzeigen

Wenn wir icons schon mit reinnehmen in das actor sheet, dann sollten wir die in der tabelle auch anzeigen
oskaloq commented 2024-01-03 11:45:22 +01:00
Author
Member
Copy Link

War auch der Plan für den nächsten Schritt. Ich habe das nur gleich mitgemacht, als ich die Icons für Container eingeführt habe ...

War auch der Plan für den nächsten Schritt. Ich habe das nur gleich mitgemacht, als ich die Icons für Container eingeführt habe ...
source/module/actors/M5Character.ts
@ -188,0 +213,4 @@
let rollable = false;
// console.log(item.system.rolls.formulas.map((p) => p.enabled));
for (let key in item.system.rolls.formulas) {
Byroks commented 2024-01-03 09:31:08 +01:00
Owner
Copy Link

Falls wir würfe rausnehmen, kann das weg

Falls wir würfe rausnehmen, kann das weg
oskaloq commented 2024-01-03 12:00:28 +01:00
Author
Member
Copy Link

Ich habe sie erst einmal absichtlich drinnen gelassen: Container sind im Prinzip auch nur Gegenstände und könnten zB genauso verzaubert sein. Auch mods habe ich erstmal drinnen gelassen. Ein magischer Rucksack könnte ja die Traglast verbessern ...

Ich habe sie erst einmal absichtlich drinnen gelassen: Container sind im Prinzip auch nur Gegenstände und könnten zB genauso verzaubert sein. Auch mods habe ich erstmal drinnen gelassen. Ein magischer Rucksack könnte ja die Traglast verbessern ...
source/module/actors/M5Character.ts
@ -188,0 +226,4 @@
calc: item.system.calc,
equipped: item.system?.equipped,
weight: item.system.weight || 0,
containerId: item.system.containerId || "",
Byroks commented 2024-01-03 09:29:17 +01:00
Owner
Copy Link

Container brauch nicht seine eigene ID als element haben, das wird schon über item.id geregelt

Container brauch nicht seine eigene ID als element haben, das wird schon über item.id geregelt
👍 1
source/module/items/M5Item.ts
@ -18,0 +20,4 @@
if (actor) {
const actorCalc = actor.derivedData({ weapons: true, defensiveWeapons: true, armor: true, items: true, spells: true, effects: true, kampfkuenste: true });
if (actorCalc) {
calc.containers = actorCalc.gear.containers;
Byroks commented 2024-01-03 09:39:54 +01:00
Owner
Copy Link

Mir gefällt nicht das wir alle container im item speichern aber ich sehe spontan leider auch keine andere möglichkeit

Mir gefällt nicht das wir alle container im item speichern aber ich sehe spontan leider auch keine andere möglichkeit
oskaloq commented 2024-01-03 12:03:08 +01:00
Author
Member
Copy Link

Die anderen Lösungen (zB DSA) machen es wohl auch ähnlich ...

Die anderen Lösungen (zB DSA) machen es wohl auch ähnlich ...
templates/sheets/item/container.hbs
@ -0,0 +27,4 @@
</div>
</td>
</tr>
<tr>
Byroks commented 2024-01-03 08:35:25 +01:00
Owner
Copy Link

Brauchen wir den Wert eines Aufbewahrungsorts? Interessant wäre es vielleicht den Wert aller gegenstände in einem Aufbewahrungsort zusammen zu rechnen. Dann könnte man das auch für den Hort verwenden.

Brauchen wir den Wert eines Aufbewahrungsorts? Interessant wäre es vielleicht den Wert aller gegenstände in einem Aufbewahrungsort zusammen zu rechnen. Dann könnte man das auch für den Hort verwenden.
oskaloq commented 2024-01-03 12:16:50 +01:00
Author
Member
Copy Link

Ich denke, wir brauchen Beides: Den Wert des Aufbewahrungsorts (so ein magischer Rucksack kann ganz schön teuer werden ;->), als auch den Gesamtwert der Aufbewahrung (Wert der Aufbewahrung selbst + enthaltene Gegenstände) ...

Ich denke, wir brauchen Beides: Den Wert des Aufbewahrungsorts (so ein magischer Rucksack kann ganz schön teuer werden ;->), als auch den Gesamtwert der Aufbewahrung (Wert der Aufbewahrung selbst + enthaltene Gegenstände) ...
templates/sheets/item/container.hbs
@ -0,0 +47,4 @@
{{> "systems/midgard5/templates/sheets/item/rolls.hbs"}}
{{> "systems/midgard5/templates/sheets/partial/mod.hbs" mods=data.mods calc=data.calc}}
Byroks commented 2024-01-03 08:34:46 +01:00
Owner
Copy Link

Ein Aufbewahrungsort braucht meiner Meinung nach keine möglichen Mods oder Rolls

Ein Aufbewahrungsort braucht meiner Meinung nach keine möglichen Mods oder Rolls
oskaloq commented 2024-01-03 12:20:19 +01:00
Author
Member
Copy Link

S.o.: Wäre aus meiner Sicht cool, wenn wir die möglichen Mods eher noch auf die Traglast des Acors bzw. das Gesamtgewicht eines Containers (inkl. enthaltene Gegenstände) erweitern würden ...

S.o.: Wäre aus meiner Sicht cool, wenn wir die möglichen Mods eher noch auf die Traglast des Acors bzw. das Gesamtgewicht eines Containers (inkl. enthaltene Gegenstände) erweitern würden ...
oskaloq commented 2024-01-03 11:42:55 +01:00
Author
Member
Copy Link

Ja, ich hatte auch schon überlegt (und testweise implementiert), ob wir Gegenstände in Containern aus der allgemeinen Liste rausnehmen sollten. Ich fand es aber gut, noch einen Ort zu haben, wo alle Gegenstände aufgeführt sind (ich würde dann noch den Titel entsprechend ändern in "Alle Gegenstände"). Wie wäre es mit einem Kompromiss: Wir nehmen sie standardmäßig raus, lassen sie User aber bei Bedarf wieder einblenden (Haken im Tabellen Header). Wäre auch gut für den Fall, dass etwas schief geht (zB ein nicht leerer Container gelöscht wird) ...
In diesem Fall würde ich dann aber nicht zwei feste Spalten machen (wie jetzt der Fall), sondern das ganze flexibler gestalten, da der allgemeine Gegenstands Container dann voraussichtlich weniger Platz in der Höhe einnehmen würde.

Ja, ich hatte auch schon überlegt (und testweise implementiert), ob wir Gegenstände in Containern aus der allgemeinen Liste rausnehmen sollten. Ich fand es aber gut, noch einen Ort zu haben, wo alle Gegenstände aufgeführt sind (ich würde dann noch den Titel entsprechend ändern in "Alle Gegenstände"). Wie wäre es mit einem Kompromiss: Wir nehmen sie standardmäßig raus, lassen sie User aber bei Bedarf wieder einblenden (Haken im Tabellen Header). Wäre auch gut für den Fall, dass etwas schief geht (zB ein nicht leerer Container gelöscht wird) ... In diesem Fall würde ich dann aber nicht zwei feste Spalten machen (wie jetzt der Fall), sondern das ganze flexibler gestalten, da der allgemeine Gegenstands Container dann voraussichtlich weniger Platz in der Höhe einnehmen würde.
Byroks commented 2024-01-03 13:25:22 +01:00
Owner
Copy Link

Ich mag deinen Vorschlag, setz das ruhig so um!
Vergiss nicht den Filter in M5Character.ts:153 anzupassen, sonst funktionieren die Mods von Containern nicht

Ich mag deinen Vorschlag, setz das ruhig so um! Vergiss nicht den Filter in M5Character.ts:153 anzupassen, sonst funktionieren die Mods von Containern nicht
Byroks added 1 commit 2024-01-03 13:57:09 +01:00
Byroks approved these changes 2024-01-03 13:58:16 +01:00
oskaloq merged commit b945f22710 into develop 2024-01-03 15:57:18 +01:00
oskaloq deleted branch oskaloq 2024-01-03 15:57:18 +01:00
oskaloq commented 2024-01-03 16:50:24 +01:00
Author
Member
Copy Link

Ich mag deinen Vorschlag, setz das ruhig so um!
Vergiss nicht den Filter in M5Character.ts:153 anzupassen, sonst funktionieren die Mods von Containern nicht

Da warst Du schneller ;->

> Ich mag deinen Vorschlag, setz das ruhig so um! > Vergiss nicht den Filter in M5Character.ts:153 anzupassen, sonst funktionieren die Mods von Containern nicht Da warst Du schneller ;->
Sign in to join this conversation.
Reviewers
No reviewers
Byroks
Labels
Clear labels
  
bug

Something is not working

  
feature

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
regelfehler

Funktionalität stimmt nicht mit Regeln überein

  
wontfix

This won't be fixed

No Label
bug
feature
help wanted
invalid
question
regelfehler
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MidgardVTT-Entwicklung/foundry-vtt-system-midgard5#9
No description provided.
Delete Branch "oskaloq"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?