Skip to content

Remove DOM Renderer Path #784

Description

@SorraTheOrc

Summary

Eliminate svgDom runtime rendering/toggles and render cards exclusively in Phaser-native layers.

Player and user experience impact

  • Player experience: Card visuals remain visible and correctly occluded by overlays using Phaser layer/depth semantics.
  • Developer experience: One consistent rendering path with reduced DOM-specific maintenance overhead.

Acceptance Criteria

  • No production path calls scene.svgDom.createOrUpdate or scene.svgDom.setVisible.
  • HelpPanel and SettingsPanel contain no svgDom hide/show toggles.
  • Main Street cards render through Phaser-native objects in all normal gameplay states.
  • Main Street browser and unit tests pass with no DOM-visibility assertions for overlays.

Minimal Implementation (end-to-end slice)

  • Remove svgDom visibility toggles from src/ui/HelpPanel.ts and src/ui/SettingsPanel.ts.
  • Remove or disable DOM render branches in Main Street renderer/lifecycle files.
  • Implement on a dedicated git branch for safe rollback via revert/merge strategy.
  • Add/adjust tests to verify behavior without DOM visibility side effects.

Dependencies

  • None

Deliverables

  • Code changes
  • Regression tests
  • Worklog implementation notes

References

  • Parent: Phaser-native rendering for DOM SVG card visuals (replace svgDom) (CG-0MP8YFNX4007N9ZU)

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions