Skip to content

Timeline: Shift+Click to seek Play Preview - #324

Open
austinscantland wants to merge 1 commit into
Reloe:mainfrom
austinscantland:feat/timeline-preview-seek
Open

austinscantland wants to merge 1 commit into
Reloe:mainfrom
austinscantland:feat/timeline-preview-seek

Conversation

@austinscantland

Copy link
Copy Markdown

What does this PR do?

Play Preview on the reminder timeline can start from any fight time, not just pull. Shift+Click the timeline (or a reminder/boss-event block) to park the playhead; Play Preview then starts from that second in the correct phase and skips reminders that have already passed.

Stop Preview parks the playhead so Play resumes from there. The playhead resets when preview reaches the end of the fight or you leave the current view. Help text is updated in enUS, koKR, ruRU, zhCN, and zhTW.

How was it tested?

In-game on live with a loaded reminder set:

  • Play Preview from 0 still behaves as before
  • Shift+Click parks the playhead; Play starts from that time in the correct phase
  • Shift+Click while preview is running jumps the playhead and reschedules remaining reminders
  • Stop parks the playhead; Play resumes from there
  • Zoom, right-drag navigate, Ctrl+Scroll, and edit-mode retime still work
  • Switching to All Reminders / View Note hides preview and resets the playhead

Checklist

  • Existing Play Preview from 0 is unchanged unless you Shift+Click to seek
  • Shift+Click does not change zoom, right-drag navigate, or edit-mode retime
  • Mid-fight preview starts in the correct phase and skips already-passed reminders
  • Help text updated in enUS, koKR, ruRU, zhCN, zhTW
  • Tested in-game on live

Contact Info

  • Discord: espyr

Made with Cursor

Preview can start mid-fight in the correct phase, skipping already-passed reminders, instead of always restarting from pull.

Co-authored-by: Cursor <cursoragent@cursor.com>
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.

1 participant