Skip to content

Comments

Fix - stale recording overlay#5411

Open
WizardOfYendor1 wants to merge 2 commits intojellyfin:masterfrom
WizardOfYendor1:fix/stale-recording-overlay
Open

Fix - stale recording overlay#5411
WizardOfYendor1 wants to merge 2 commits intojellyfin:masterfrom
WizardOfYendor1:fix/stale-recording-overlay

Conversation

@WizardOfYendor1
Copy link
Contributor

@WizardOfYendor1 WizardOfYendor1 commented Feb 11, 2026

When you press up on the remote (overlay) when watching livetv then press the record button, the button doesn't change state (e.g. doesn't change red). You have to leave the menu then come back to see if it's recording. Same is true if you cancel the recording.

On a side note, there are psuedo related issues with #5396 and jellyfin/jellyfin#16191.... (which is why I keep peeling the onion :-) ). You would need 16191 or live in a + UTC to even be able to press record in the first place from the overlay. Once I could do that (-5 UTC here), I noticed all these other problems and started at least trying to fix them.

Changes
Change to update the "items" list dynamically so the user doesn't have to exit and come back to get a refresh.
The "big" change was to making the items list mutable on the setter call.

Opus 4.6 was used to check potential impacts it the "ecosystem" and research tasks. There were none noted.

@WizardOfYendor1 WizardOfYendor1 force-pushed the fix/stale-recording-overlay branch 3 times, most recently from 5cf70ea to b4f5925 Compare February 11, 2026 23:17
@WizardOfYendor1 WizardOfYendor1 marked this pull request as ready for review February 12, 2026 02:07
@WizardOfYendor1
Copy link
Contributor Author

This fix it part of the myriad of PRs I have out there dealing with the display overlay.

I apologize for how scatterbrained it may be but I'd fix one thing only discover that it exposed another issue. Fix that and so forth and so one.

I have merged them all up into an integration branch I have in my fork that I use to build and test them (on a Fire Cube). Seems to be behaving and working. It's definitely no worse for me.

Copy link
Member

@nielsvanvelzen nielsvanvelzen left a comment

Choose a reason for hiding this comment

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

The mItems should be read-only. We cannot change it unless we replace all items, which cannot be done during playback.

@WizardOfYendor1
Copy link
Contributor Author

The mItems should be read-only. We cannot change it unless we replace all items, which cannot be done during playback.

Hmmmm... OK. I'm curious as to what problem that creates but I'll take your word for it. Oddly, I was trying a different approach that I thought was uglier I'll see if I can get that working and make mItems immutable again.

@WizardOfYendor1 WizardOfYendor1 force-pushed the fix/stale-recording-overlay branch 2 times, most recently from a3546b9 to d1472f2 Compare February 13, 2026 17:32
@WizardOfYendor1 WizardOfYendor1 changed the title Fix/stale recording overlay Fix - stale recording overlay Feb 15, 2026
@WizardOfYendor1
Copy link
Contributor Author

I've made changes to restore mItems to being immutable again. I used sorta a "shadow object" accomplish the same effect. There maybe a better approach.

I detected another issue as noted in the subsequent commit that is related. This one is, to me, more straight forward.

Lastly, there are issues surrounding recording a series, cancelling a series, then recording the series again. Issues that have existed before any of this. This is what I consider an edge case (i.e. unrealistic), as I'm blasting through testing stuff. I've got some preliminary analysis but I think it's going to involve issues with the server side. It's a deeper rabbit hole, I don't have time to address right now.

If there is better approaches, or changes, I will not be offended if the maintainers wish to make them in-situ of these commits/PR(s) :-)

…el, then sets the next program to record from the guide, then resumes watching the channel. When the current program ends, and the next (recorded) starts, the recording state will not correctly show it is being recorded.
@WizardOfYendor1 WizardOfYendor1 force-pushed the fix/stale-recording-overlay branch from db4770f to 082c2be Compare February 21, 2026 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants