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

Add option to hide OSD message #45

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dariusjonda
Copy link

@dariusjonda dariusjonda commented Oct 1, 2021

referring to the feature request in issue #44 I've added the osd_text_show option to choose whether to show the OSD message or not whenever a SponsorBlock action is being triggered.
The function osd_show() has been added to return the actual SponsorBlock text in case ost_text_show = true and an empty string when set to false.

I've never done anything in Lua before, therefore my simple solution might not be the most elegant one.
However I hope this is kinda helpful...

in order to hide the text that shows up whenever a sponsoblock
action has been triggered, a function has been added, that
returns an empty string whenever the option "osd_text_show" is
set to false
see issue po5#44
@po5
Copy link
Owner

po5 commented Oct 5, 2021

This is a bad hack that will display empty messages instead of nothing at all.
It will play badly with other scripts displaying their own messages.
These mp.osd_message() calls shouldn't happen.

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