Skip to content

refactor: various logic improvements#531

Merged
Samillion merged 3 commits intomainfrom
refactor_various_logic
Mar 13, 2026
Merged

refactor: various logic improvements#531
Samillion merged 3 commits intomainfrom
refactor_various_logic

Conversation

@Samillion
Copy link
Owner

Changes:

  • Change screenshot button default value to only show video
  • Cache hover effects instead of calls on every frame
  • Move format_time() and build_cache_seek_ranges outside of osc_init()
  • Use already cached state.chapter_list instead of re-calling property various times
  • Use mp.get_property_bool() instead of redundant string check mp.get_property() == yes/no
  • Remove pause_state() function, didn't need to exist in the first place
  • Simplify mouse init and pos checks in process_event()
  • Reset state.visibility_modes before populating
  • Don't retrieve display-fps at startup as it doesn't work.
    • follows upstream mpv

@Samillion Samillion merged commit 2388e43 into main Mar 13, 2026
@Samillion Samillion deleted the refactor_various_logic branch March 13, 2026 01:15
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.

1 participant