thepaperpilot/Profectus
1
0
Fork 0
forked from profectus/Profectus
Code Pull requests Activity Actions

Fix smallerIsBetter handling in createCollapsibleModifierSections

Browse source
This commit is contained in:
thepaperpilot 2023-04-16 12:43:04 -05:00
parent 1928be236d
commit 8ebfb85360
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/data/common.tsx
View file

@ -253,17 +253,17 @@ export interface Section {
baseText?: Computable<CoercableComponent>;
/** Whether or not this section should be currently visible to the player. **/
visible?: Computable<boolean>;
/** Determines if numbers larger or smaller than the base should be displayed as red. */
smallerIsBetter?: boolean;
}
/**
* Takes an array of modifier "sections", and creates a JSXFunction that can render all those sections, and allow each section to be collapsed.
* Also returns a list of persistent refs that are used to control which sections are currently collapsed.
* @param sectionsFunc A function that returns the sections to display.
* @param smallerIsBetter Determines whether numbers larger or smaller than the base should be displayed as red.
*/
export function createCollapsibleModifierSections(
sectionsFunc: () => Section[],
smallerIsBetter = false
sectionsFunc: () => Section[]
): [JSXFunction, Persistent<Record<number, boolean>>] {
const sections: Section[] = [];
const processed:
@ -324,7 +324,9 @@ export function createCollapsibleModifierSections(
{s.unit}
</span>
</div>
{renderJSX(unref(s.modifier.description))}
{s.modifier.description == null
? null
: renderJSX(unref(s.modifier.description))}
</>
);
@ -353,7 +355,7 @@ export function createCollapsibleModifierSections(
class="modifier-amount"
style={
(
smallerIsBetter === true
s.smallerIsBetter === true
? Decimal.gt(total, base ?? 1)
: Decimal.lt(total, base ?? 1)
)