-
Notifications
You must be signed in to change notification settings - Fork 22
UI overhaul i feel the rush #1383
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
base: main
Are you sure you want to change the base?
Conversation
Fixed encounter member column runtime reference to columns. Added resize observation to encounter members tab. Fixed CR column. Fixed Quantity column.
…caching context services are missing.
…, instead of item-specific ones.
…acker and build atop it.
- [x] Row 1 - [x] Creature Type Count row - [x] Row 2 - [x] Members count (assuming member quantities sum) - [x] XP - [x] GP summary - [x] Encounter header layout change - [x] Left: name, subtitles, button bar - [x] Right: difficulty badge
…lti-actor base class to share.
Set up encounter sidebar with Languages, Speeds, Senses, Skills, and Specials. Pulled back more content to share between multi-actor sheets. Removed draggability and context menu options from GroupSkillsCard.
Extend some group member tab styles to the combat tab. Removed rogue divider dot on encounter member name subtitle. Added members table to combat tab.
…and members tabs. Created encounter member toggle row action. Set up row actions for combat tab. Created ActivityActionsColumnHeader for adding activities. Corrected tooltip for the item create button.
…for Encounter members who are editable.
Added Quantity Formula column to Encounter Members table.
Encounter sheet style updates
…-tidy5e-sheet-2 into ui-overhaul-i-feel-the-rush
…make that the flag. Likewise, put all combat settings into a combat prop on the Member and Placeholder contexts. - [x] Refactor: ensure all encounter combat data flag updates also trim away nonexistent members/placeholders every time there's an update.
Removed unused code.
cubicFoot: VolumeUnitConfig; | ||
liter: VolumeUnitConfig; | ||
} & Record<string, VolumeUnitConfig>; | ||
ENCOUNTER_DIFFICULTY: [number, number, number][]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick: I realise this comes from the dnd5e module, but I kinda dislike unlabeled tuples.
ENCOUNTER_DIFFICULTY: [number, number, number][]; | |
ENCOUNTER_DIFFICULTY: [low: number, medium: number, high: number][]; |
would be a slight improvement just for the sake of readibility
document={context.actor} | ||
field={bioField.field} | ||
value={bioField.value} | ||
selectOnFocus={true} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Opinion: I believe you could just say selectOnFocus
without explicitly saying it's true
. It would match regular html as well (e.g <input disabled required>
</div> | ||
<ul class="biography-entries"> | ||
{#each bioFields as bioField (bioField.field)} | ||
{#if bioField.value != null && bioField.value !== '' || context.unlocked} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you care (and is it possible) for the value to just be whitespace? i.e do you need to ?.trim()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't put trim since you can also have empty tags like <h1></h1>
that you'd also have to look for and trim in that case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmmm. potentially possible with a utility function I guess 😅 apply innerHTML
and read textContent
{#if (context.enriched.appearance !== '' && | ||
context.enriched.trait !== '' && | ||
context.enriched.ideal !== '' && | ||
context.enriched.bond !== '' && | ||
context.enriched.flaw !== '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thought: Not sure if this would work out better as an array with a .every
…t when creating a new placeholder.
…set placeholders (still working on images).
…-vtt-tidy-5e-sheets into ui-overhaul-i-feel-the-rush
…cells. Fix for text alignment in Character sheet tables.
Implemented image preview and filepicker image selection on Encounter Combat tab placeholders.
Implemented difficulty target swapping which applies to all encounter sheets from the user's perspective.
… all. Added style block for combat difficulty target select.
In addition to overhaul work: