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

Add CONVERT_COORD_TO_JAGCOORD setting #47

Merged
merged 2 commits into from
Feb 11, 2025
Merged

Add CONVERT_COORD_TO_JAGCOORD setting #47

merged 2 commits into from
Feb 11, 2025

Conversation

Tomm0017
Copy link
Collaborator

Before
Before

After
After

  • Adds IdentifiedChildProperty as a base class for identified entity properties.

  • Each IdentifiedChildProperty has its own formattedValue function, which takes in a SettingSet and SymbolDictionary to allow for flexibility and future maintainability.

  • Replaces the old PropertyFormatterCollection npc property formatters.

  • Adds a NopSettingSet and NopSymbolDictionary to maintain consistency with the old value property each of these new IdentifiedChildPropertys previously had. These nop variants are not used anywhere outside the value assignment.

  • Adds the Setting.CONVERT_COORD_TO_JAGCOORD set to disabled by default.

- Adds `IdentifiedChildProperty` as a base class for identified
entity properties.

- Each `IdentifiedChildProperty` has its own `formattedValue`
function, which takes in a `SettingSet` and `SymbolDictionary` to
allow for flexibility and future maintainability.

- Replaces the old `PropertyFormatterCollection` npc property
formatters.

- Adds a `NopSettingSet` and `NopSymbolDictionary` to maintain
consistency with the old `value` property each of these new
`IdentifiedChildProperty`s previously had. These nop variants
are not used anywhere outside the `value` assignment.

- Adds the `Setting.CONVERT_COORD_TO_JAGCOORD` set to disabled by
default.
@Z-Kris Z-Kris merged commit e398174 into master Feb 11, 2025
4 checks passed
@Z-Kris Z-Kris deleted the feat/jag-coords branch February 11, 2025 20:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants