mirror of
https://github.com/aome510/spotify-player.git
synced 2026-04-26 09:25:49 +03:00
[PR #274] [MERGED] Clamped progress bars to solve problems with -ve numbers #698
Labels
No labels
bug
documentation
enhancement
good first issue
help wanted
pull-request
question
third-party
third-party
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/spotify-player#698
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/aome510/spotify-player/pull/274
Author: @whiskyplausible
Created: 10/22/2023
Status: ✅ Merged
Merged: 10/23/2023
Merged by: @aome510
Base:
master← Head:master📝 Commits (2)
1b396eaClamped progress bars to solve problems with -ve numbers30c40fbapply cargo fmt📊 Changes
1 file changed (+7 additions, -2 deletions)
View changed files
📝
spotify_player/src/ui/playback.rs(+7 -2)📄 Description
Resolves #273
Negative numbers were sometimes appearing in progress.num_seconds() which would cause a panic, as the ui elements would only accept positive ratios. Clamped the numbers from 0.0 to 1.0 to prevent this happening. Seems to have solved the panics, but no idea where the negative numbers are coming from.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.