Skip to content
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

refactor: don't show osc on start when osc_on_seek=yes #111

Merged
merged 1 commit into from
Oct 26, 2024

Conversation

Samillion
Copy link
Owner

No description provided.

@Samillion Samillion linked an issue Oct 26, 2024 that may be closed by this pull request
@Samillion Samillion merged commit 3cb9293 into main Oct 26, 2024
@Samillion Samillion deleted the dev_osconseek_refactor branch October 26, 2024 22:19
@Keith94
Copy link
Contributor

Keith94 commented Oct 26, 2024

I can't get it to work :( any ideas?

@Samillion
Copy link
Owner Author

Are you sure? The behavior now is that it doesn't show OSC on start/next file, only if the OSC already showing it will not explicitly hide it, it'll just rely on hidetimeout then.

osc_on_seek.mp4

@Keith94
Copy link
Contributor

Keith94 commented Oct 26, 2024

Oh sorry you're right, it's my mistake. Works beautifully now!

@Keith94
Copy link
Contributor

Keith94 commented Oct 26, 2024

Okay I just realized why it wasn't working for me.

If you have save-position-on-quit enabled, load and quit a file, then load the same file, the osc shows on startup because the player is seeking to the last position it was on before quitting.

Can ModernZ detect that?

@Samillion
Copy link
Owner Author

I'll have to think about that one. Since technically with that scenario, it literally seeks back to position. So the event "seeking" is triggered.

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.

Don't show osc when file loaded if osc_on_seek=yes
2 participants