Skip to content

[#4708] Deprecate AdvancementSelection in favor of createDialog#5228

Merged
arbron merged 2 commits into
4.4.xfrom
advancement/creation-dialog
Mar 21, 2025
Merged

[#4708] Deprecate AdvancementSelection in favor of createDialog#5228
arbron merged 2 commits into
4.4.xfrom
advancement/creation-dialog

Conversation

@arbron

@arbron arbron commented Feb 19, 2025

Copy link
Copy Markdown
Collaborator

Deprecates the V1 AdvancementSelection dialog in favor of using PseudoDocument#createDialog same as activities. To fully support using advancements with this method a few changes were made:

  • Added support for proper coloring of SVG icons
  • Added optional hint that is added as tooltip
  • Added ability to disable options without removing them
  • Split some logic out from the createDialog method into _createDialogTypes and _createDialogData methods so each type can customize the dialog behavior
Advancement Creation Dialog

Deprecates the V1 `AdvancementSelection` dialog in favor of using
`PseudoDocument#createDialog` same as activities. To fully support
using advancements with this method a few changes were made:
- Added support for proper coloring of SVG icons
- Added optional `hint` that is added as tooltip
- Added ability to disable options without removing them
- Split some logic out from the `createDialog` method into
  `_createDialogTypes` and `_createDialogData` methods so each
  type can customize the dialog behavior
@arbron arbron added ui User interface related features or bugs system: advancement labels Feb 19, 2025
@arbron arbron added this to the D&D5E 4.4.0 milestone Feb 19, 2025
@arbron arbron self-assigned this Feb 19, 2025
@arbron
arbron changed the base branch from 4.3.x to 4.4.x February 24, 2025 17:31
@Fyorl
Fyorl force-pushed the 4.4.x branch 2 times, most recently from ecdba9c to 52777c2 Compare March 5, 2025 15:07
@Fyorl
Fyorl force-pushed the 4.4.x branch 2 times, most recently from 338f711 to 6710279 Compare March 21, 2025 14:38
Comment thread module/documents/mixins/pseudo-document.mjs Outdated
@arbron
arbron merged commit e4d0381 into 4.4.x Mar 21, 2025
@arbron
arbron deleted the advancement/creation-dialog branch March 21, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

system: advancement ui User interface related features or bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants