Skip to content

refactor: clean up checklist structure + node renderer - #7073

Open
IMB11 wants to merge 2 commits into
mainfrom
cal/fix-moderation-pkg
Open

refactor: clean up checklist structure + node renderer#7073
IMB11 wants to merge 2 commits into
mainfrom
cal/fix-moderation-pkg

Conversation

@IMB11

@IMB11 IMB11 commented Aug 10, 2026

Copy link
Copy Markdown
Member
  • Applies component standards to the checklist + node renderer components
  • Node renderer has moved back into apps/frontend
  • Node renderer now works like:
type NodeRendererDescriptor =
	| { type: 'action' | 'checkbox' | 'toggle' | 'dropdown' | 'text' | 'markdown' }
	| { type: 'custom'; key: string }

/// ...

withRenderer(node, {
	renderer: { type: 'action' },
})

/// then frontend maps the type to an actual component:

const builtinRenderers = {
	action: { component: ActionButton, ... },
	checkbox: { component: Checkbox, ... },
	toggle: { component: Toggle },
	dropdown: { component: Combobox, ... },
	text: { component: StyledInput, ... },
	markdown: { component: MarkdownEditor, ... },
}

this is in line with how we do it for the server panel's audit log table, which has lots of data defined ui in the table

@IMB11
IMB11 requested a review from chyzman August 10, 2026 11:00
@modrinth-bot

Copy link
Copy Markdown
Member

Pull request changelog

App

Added

Changed

Deprecated

Removed

Fixed

Security

Website

Added

Changed

Deprecated

Removed

Fixed

Security

Hosting

Added

Changed

Deprecated

Removed

Fixed

Security

@modrinth-bot

modrinth-bot commented Aug 10, 2026

Copy link
Copy Markdown
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants