Skip to content

Commit

Permalink
Fix aspect settings + mergeObejct
Browse files Browse the repository at this point in the history
  • Loading branch information
Plenett committed Jan 29, 2025
1 parent a41ed6f commit 20ac7ca
Show file tree
Hide file tree
Showing 5 changed files with 20 additions and 11 deletions.
9 changes: 9 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,14 @@
# Patch Notes:

## Version 1.4.1

* Fix aspect settings related to font #25
* Fix `mergeObject` deprecated

## Version 1.4.0

* Update for V12

## Version 1.3.0

* Remove `_sortable.js` from module dependcy to local dependency #22
Expand Down
2 changes: 1 addition & 1 deletion module.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"id": "fate-aspect-tracker",
"title": "Fate Aspect Tracker",
"description": "An Aspect Tracker for any Fate RPG.",
"version": "1.4.0",
"version": "1.4.1",
"authors": [
{
"name": "Théo Plénet",
Expand Down
2 changes: 1 addition & 1 deletion modules/fate-aspect-tracker.js
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ Hooks.once('init', async() => {
config: false,
type:String,
restricted:true,
choices:CONFIG.fontFamilies,
choices:Object.keys(CONFIG.fontDefinitions),
default:"Arial",
});

Expand Down
4 changes: 2 additions & 2 deletions modules/tracker.js
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ export class Tracker {
strokeAlpha: game.settings.get("fate-aspect-tracker","AspectDrawingBorderOpacity"),
text: text,
fontSize: fontsize,
fontFamily: CONFIG.fontDefinitions[game.settings.get("fate-aspect-tracker","AspectDrawingFontFamily")],
fontFamily: game.settings.get("fate-aspect-tracker","AspectDrawingFontFamily"),
textColor: color,
points: []
};
Expand Down Expand Up @@ -273,7 +273,7 @@ export class Tracker {
}).filter(d => d != null);

scene.updateEmbeddedDocuments('Drawing', drawings);
newDrawings = newDrawings.concat(drawings.map(drawing => drawing._id));
newDrawings = newDrawings.concat(drawings.map(drawing => drawing._id));
});

// Replace drawings with existing textbox (i.e. Remove from list deleted textbox)
Expand Down
14 changes: 7 additions & 7 deletions modules/ui.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/* global jQuery, Handlebars, Sortable */
/* global game, loadTemplates, mergeObject, Application, FormApplication, Dialog */
/* global game, loadTemplates, Application, FormApplication, Dialog */

import { Aspect, Tracker } from "./tracker.js";
import { RGBColor } from "./colors.js";
Expand Down Expand Up @@ -40,7 +40,7 @@ async function preloadTemplates() {

export class AspectTrackerWindow extends Application {
static get defaultOptions() {
return mergeObject(super.defaultOptions, {
return foundry.utils.mergeObject(super.defaultOptions, {
id: "fate-aspect-tracker-app",
template: "modules/fate-aspect-tracker/templates/aspect-list.hbs",
width: 400,
Expand Down Expand Up @@ -206,7 +206,7 @@ export class AspectTrackerWindow extends Application {

class AspectForm extends FormApplication {
static get defaultOptions() {
return mergeObject(super.defaultOptions, {
return foundry.utils.mergeObject(super.defaultOptions, {
id: "fate-aspect-tracker-form",
template: "modules/fate-aspect-tracker/templates/aspect-item-form.hbs",
width: 400,
Expand Down Expand Up @@ -262,7 +262,7 @@ class AspectForm extends FormApplication {

class AspectDrawingSettings extends FormApplication {
static get defaultOptions() {
return mergeObject(super.defaultOptions, {
return foundry.utils.mergeObject(super.defaultOptions, {
id: "fate-aspect-drawing-settings",
template: "modules/fate-aspect-tracker/templates/aspect-drawing-settings.hbs",
width: 450,
Expand All @@ -288,8 +288,8 @@ class AspectDrawingSettings extends FormApplication {
/** @override */
getData() {
return {
fontFamilies:CONFIG.fontFamilies,
fontFamily:CONFIG.fontFamilies[game.settings.get("fate-aspect-tracker","AspectDrawingFontFamily")],
fontFamilies:Object.keys(CONFIG.fontDefinitions),
fontFamily:game.settings.get("fate-aspect-tracker","AspectDrawingFontFamily"),
fontSize:game.settings.get("fate-aspect-tracker","AspectDrawingFontSize"),
fontDynamicColor:game.settings.get("fate-aspect-tracker","AspectDrawingFontDynamicColor"),
fontColor:game.settings.get("fate-aspect-tracker","AspectDrawingFontColor"),
Expand All @@ -314,7 +314,7 @@ class AspectDrawingSettings extends FormApplication {
let borderColor = data.border_color;
let borderOpacity = data.border_opacity;

await game.settings.set("fate-aspect-tracker","AspectDrawingFontFamily", CONFIG.fontFamilies.indexOf(fontFamily));
await game.settings.set("fate-aspect-tracker","AspectDrawingFontFamily", fontFamily);
await game.settings.set("fate-aspect-tracker","AspectDrawingFontSize", fontSize);
await game.settings.set("fate-aspect-tracker","AspectDrawingFontDynamicColor", fontDynamicColor);
await game.settings.set("fate-aspect-tracker","AspectDrawingFontColor", fontColor);
Expand Down

0 comments on commit 20ac7ca

Please sign in to comment.