Skip to content

feat(settings): consolidate inline mechanics toggles into single setting - #645

Merged
zkat merged 1 commit into
iron-vault-plugin:mainfrom
nickarrow:feature/consolidate-inline-render-toggle
Jan 30, 2026
Merged

zkat merged 1 commit into
iron-vault-plugin:mainfrom
nickarrow:feature/consolidate-inline-render-toggle

Conversation

@naretakis

Copy link
Copy Markdown
Contributor

Replace 8 individual inline render settings with a single useInlineMechanics toggle that controls all inline formatting at once.

Removed settings:

  • useInlineMoves
  • useInlineOracles
  • useInlineProgressTracks
  • useInlineClocks
  • useInlineMeters
  • useInlineEntities
  • useInlineDiceRolls
  • useInlineOOC

Hand spun changes by me.

BREAKING CHANGE: Users who had individual inline settings enabled will need to re-enable the new consolidated setting.

Replace 8 individual inline render settings with a single `useInlineMechanics`
toggle that controls all inline formatting at once.

Removed settings:
- useInlineMoves
- useInlineOracles
- useInlineProgressTracks
- useInlineClocks
- useInlineMeters
- useInlineEntities
- useInlineDiceRolls
- useInlineOOC

BREAKING CHANGE: Users who had individual inline settings enabled will
need to re-enable the new consolidated setting.
@naretakis

Copy link
Copy Markdown
Contributor Author

@zkat I tested this locally and it seems to work great. But it is indeed a breaking change, as it will revert to the default of blocks. Users just have to go back in and enable the single toggle in settings, but just calling that out.

Likewise, we really don't know if anyone has partial toggles on for the inline stuff, so this could disrupt some players' experiences. Let me know what you think.

I'd say just push it out and if folks report back that they don't like it, we could always revert.

As always, if there is anything odd here, let me know. Trying to keep up y'all's good practices.

@zkat

zkat commented Jan 30, 2026

Copy link
Copy Markdown
Member

I think inline mechanics are new and experimental enough that we can just give people a heads-up directly in the channel

@zkat
zkat merged commit c181a00 into iron-vault-plugin:main Jan 30, 2026
4 checks passed
@zkat zkat added the feature New feature or request label Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants