[Refactor] Update initialization #28

Closed
opened 2026-01-30 21:22:58 +00:00 by claunia · 2 comments
Owner

Originally created by @mnadareski on GitHub (Jul 10, 2021).

Currently, initialization acts like both initialize and load. These pieces of functionality should be separated out. This would allow for some parts of the state to maintain consistency, like the current playback volume.

Originally created by @mnadareski on GitHub (Jul 10, 2021). Currently, initialization acts like both initialize and load. These pieces of functionality should be separated out. This would allow for some parts of the state to maintain consistency, like the current playback volume.
Author
Owner

@mnadareski commented on GitHub (Jul 10, 2021):

Notes to self:

  • Make initialization just init of internal state
  • Make loading of images decoupled from init
  • Make more settings persistent, such as auto-play, loading tracks, etc
  • Make current playback volume consistent if loading a new image
@mnadareski commented on GitHub (Jul 10, 2021): Notes to self: - Make initialization just init of internal state - Make loading of images decoupled from init - Make more settings persistent, such as auto-play, loading tracks, etc - Make current playback volume consistent if loading a new image
Author
Owner

@mnadareski commented on GitHub (Jul 10, 2021):

Open question: What should care about autoplay? Currently, everything down to the sound output cares about autoplay. Should only the Player care? The PlayerViewModel? The PlayerView?

@mnadareski commented on GitHub (Jul 10, 2021): Open question: What should care about autoplay? Currently, everything down to the sound output cares about autoplay. Should only the Player care? The PlayerViewModel? The PlayerView?
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aaru-dps/RedBookPlayer#28