[Feature] User can toggle song info panel #67
Labels
enhancement
New feature or request
Comments
xxxserxxx
added a commit
to xxxserxxx/stmps
that referenced
this issue
Oct 16, 2024
xxxserxxx
added a commit
that referenced
this issue
Oct 16, 2024
Merged
spezifisch
pushed a commit
that referenced
this issue
Oct 16, 2024
* Changes program exit function to properly close the asset queue. Closing the queue is not currently _required_, but if we replace the queue with an on-disk queue we may need to clean up resources; this commit ensures that doesn't get missed in the future. * feat: #67, user can disable song info panel * feat: #67, updates readme for how to disable the panel
xxxserxxx
added a commit
that referenced
this issue
Oct 29, 2024
* Changes program exit function to properly close the asset queue. Closing the queue is not currently _required_, but if we replace the queue with an on-disk queue we may need to clean up resources; this commit ensures that doesn't get missed in the future. * feat: #67, user can disable song info panel * feat: #67, updates readme for how to disable the panel
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In particular, if the song info panel isn't visible, we should skip loading other song assets, such as album art. This would allow the user to limit use of more resource-heavy features, without descending into configuration hell, and also keep stmps usable on smaller displays.
Implementation-wise, this should be both a hotkey toggle and a configuration option. I think the latter will be more useful, as I suspect users will more likely want it either on all the time, or off all the time, rather than wanting to pop it up and down.
The text was updated successfully, but these errors were encountered: