Skip to content

Add optional compact parameter for party embed display#116

Draft
Claude wants to merge 2 commits intomainfrom
claude/make-embed-inline-field-optional
Draft

Add optional compact parameter for party embed display#116
Claude wants to merge 2 commits intomainfrom
claude/make-embed-inline-field-optional

Conversation

@Claude
Copy link
Copy Markdown
Contributor

@Claude Claude AI commented Mar 12, 2026

Party embeds now support configurable inline field display via an optional compact boolean parameter. By default, parties use non-compact mode (stacked fields) for better readability.

Changes

  • Removed global constant: Eliminated EMBED_FIELD_INLINE in favor of per-party configuration
  • Added compact field: Party data structure now includes compact: bool (default: False)
  • Updated embed rendering: create_party_embed() reads party.get("compact", False) to control inline display
  • Extended command interface: party create accepts optional compact parameter
    # Non-compact (default)
    [p]party create "Raid Night" "Tank Healer DPS"
    
    # Compact mode
    [p]party create "Raid Night" "Tank Healer DPS" True
  • Preserved backward compatibility: Existing parties without the field default to non-compact mode

All party creation paths (command, modal, template) properly initialize the compact field.

- Removed global EMBED_FIELD_INLINE constant
- Added 'compact' boolean field to party data structure (default: False)
- Updated create_party_embed to use party['compact'] for inline field setting
- Added optional compact parameter to party_create command
- Updated CreatePartyModal, party_template_use, and party_list to use compact setting
- Default behavior is NOT compact (inline=False) for better readability

Co-authored-by: psykzz <1134201+psykzz@users.noreply.github.com>
@Claude Claude AI changed the title [WIP] Add optional embed inline field parameter for party cog Add optional compact parameter for party embed display Mar 12, 2026
@Claude Claude AI requested a review from psykzz March 12, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants