Skip to content

Conversation

@Luke-Schreiber
Copy link

@Luke-Schreiber Luke-Schreiber commented Jan 2, 2026

Does this PR close any open issues?

Closes #208

Give a longer description of what this PR addresses and why it's needed

  • Forward and backward playback (and pause) of images in images view toolbar
  • Forward and backward playback ( and pause) of images in images view sidebar
  • Adjust FPS of video playback [slow, medium, fast]
  • Layout of images view sidebar updated

Provide pictures/videos of the behavior before and after these changes (optional)

(Ignore flashing issues, this is being addressed in a separate PR)

Screen.Recording.2026-01-20.at.11.30.26.AM.mov

TODO:
[ ] - Images have trouble loading quickly.

@Luke-Schreiber Luke-Schreiber added the frontend Deals with Vue Frontend label Jan 2, 2026
@Luke-Schreiber Luke-Schreiber marked this pull request as ready for review January 6, 2026 03:02
Copy link

@Dev-Lan Dev-Lan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • remove backwards play
  • switch forward back icons with something more distinct from triangle
  • switch FPS config UI to speedup/slowdown UI, or discrete speed options, slow, medium, fast.
  • brainstorm out of sync layers, should advancing only happen after all layers have been synced?

@Luke-Schreiber Luke-Schreiber requested review from Dev-Lan and removed request for JackWilb January 20, 2026 17:56
@Luke-Schreiber Luke-Schreiber marked this pull request as draft January 20, 2026 17:57
@Luke-Schreiber
Copy link
Author

remove backwards play
switch forward back icons with something more distinct from triangle

Done here c40b2c7

@Luke-Schreiber
Copy link
Author

Luke-Schreiber commented Jan 20, 2026

switch FPS config UI to speedup/slowdown UI, or discrete speed options, slow, medium, fast.

Done 49ab71b

@Luke-Schreiber
Copy link
Author

brainstorm out of sync layers, should advancing only happen after all layers have been synced?

I think this should be addressed in separate PR, due to adding complexity to sync logic.

@Luke-Schreiber Luke-Schreiber marked this pull request as ready for review January 20, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend Deals with Vue Frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cell Images View Playback Functionality

3 participants