Skip to content

Guide: Rotation page previews the build picked on the Talents page - #13

Merged
crazyace merged 1 commit into
mainfrom
claude/project-thread-9j2l8c
Oct 4, 2026
Merged

crazyace merged 1 commit into
mainfrom
claude/project-thread-9j2l8c

Conversation

@crazyace

@crazyace crazyace commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Requested by Jeff · project thread

Before: picking a build on the Talents page changed only the tree and the plan. The Rotation page kept showing the rotation for your current talents, so picking Combat on an Assassination rogue showed a Combat tree beside an Assassination rotation.

After: when you pick a build your talents don't already match, the Rotation page plays that build with your current talent points. That covers its ranks, the spells it teaches (Blade Flurry, Mutilate, Cold Blood and so on), and the spec of the tree it fills most. A line at the top says it's a preview and names the spec your own talents give. With nothing picked, or with the build you actually follow picked, the page shows your own talents as before. The in-combat icon always follows your real talents.

How: Rogue.Preview(s, build, points) returns a copy of the state with the build's ranks and talent-taught spells, plus the spec. Guide.Build uses it only when ns.db.guideBuild is set, still fits the mode, and differs from your ranks. Tests cover the preview line, the subtitle spec, the no-preview cases and Preview itself. Luacheck and tests/battlewright_test.py pass locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_018oLw72G7E9REdYdvv742k1


Generated by Claude Code

When you pick a build on the Talents page that your talents don't already
match, the Rotation page shows that build played with your current points
(its ranks, the spells it teaches, the spec of the tree it fills most) and
says so in a line at the top. With nothing picked, or the build you follow,
it shows your own talents as before. The in-combat icon is unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018oLw72G7E9REdYdvv742k1
@crazyace crazyace self-assigned this Oct 4, 2026
@crazyace
crazyace merged commit 37e96e7 into main Oct 4, 2026
4 checks passed
crazyace added a commit that referenced this pull request Oct 4, 2026
Merge pull request #13 from crazyace/claude/project-thread-9j2l8c
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