-
-
Notifications
You must be signed in to change notification settings - Fork 799
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix image stuttering. #3035
Fix image stuttering. #3035
Conversation
@@ -64,8 +64,9 @@ func (api *Router) GetCoverArt(w http.ResponseWriter, r *http.Request) (*respons | |||
p := req.Params(r) | |||
id, _ := p.String("id") | |||
size := p.IntOr("size", 0) | |||
square := p.BoolOr("square", false) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"square"
is not a parameter for Subsonic or OpenSubsonic getCoverArt. Probably can just pass false directly
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh... looks like the web UI uses the subsonic api to fetch images?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I proposed this. This is a quick "hack" to fix an issue in the (current) WebUI.
I don't mean to document and/or propose this to OpenSubsonic, it is just an internal thing.
For the new UI we will introduce a proper native artwork endpoint, and this parameter will be removed from the getCoverArt
endpoint.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good, please see my comments below
Download the artifacts for this pull request: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice tests. A couple of nitpicks and we should be good to merge
Fix AlbumGrid shaking when a non-square album cover is rendered.