Skip to content

refactor(tour): promote shared tour-core.js helper scaffolding#5760

Open
UmutCansinTorgayli wants to merge 1 commit into
odysseus-dev:devfrom
UmutCansinTorgayli:refactor/tour-core-helper
Open

refactor(tour): promote shared tour-core.js helper scaffolding#5760
UmutCansinTorgayli wants to merge 1 commit into
odysseus-dev:devfrom
UmutCansinTorgayli:refactor/tour-core-helper

Conversation

@UmutCansinTorgayli

@UmutCansinTorgayli UmutCansinTorgayli commented Jul 26, 2026

Copy link
Copy Markdown

Summary

Promote shared onboarding tour scaffolding (styles injection, DOM element highlight halos, tooltip placement math, HTML character streaming, and tour state cancellation) into a reusable static/js/tour-core.js helper module as requested in ROADMAP.md.

Target branch

  • This PR targets dev, not main.

Linked Issue

Part of ROADMAP.md refactor targets #5760.

Type of Change

  • Refactor / cleanup (behaviour unchanged)

Checklist

  • I searched existing issues and pull requests before opening this PR.

How to Test

  1. Run JS syntax verification via node --check static/js/tour-core.js
  2. Run Python test suite via python -m py_compile tests/test_tour_core_js.py
  3. Launch the app locally and trigger /tour from the chat composer to verify step transitions and highlight halos.

@github-actions github-actions Bot added needs work PR description incomplete — please update before review ready for review Description complete — ready for maintainer review and removed needs work PR description incomplete — please update before review labels Jul 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review Description complete — ready for maintainer review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant