Skip to content

Add option to hide OSD message#45

Open
dariusjonda wants to merge 1 commit intopo5:masterfrom
dariusjonda:feature/hide-osd-message
Open

Add option to hide OSD message#45
dariusjonda wants to merge 1 commit intopo5:masterfrom
dariusjonda:feature/hide-osd-message

Conversation

@dariusjonda
Copy link
Copy Markdown

@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
@dariusjonda dariusjonda force-pushed the feature/hide-osd-message branch from 38aaa36 to ec4444c Compare October 1, 2021 17:27
@po5
Copy link
Copy Markdown
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