PM4K / PlexMod for Kodi (18, 19, 20, 21, 22+)

Yeah, to be clear all previous screenshots before the latest did not have a “Name” tag set for the subtitle being used. Hadn’t realized it was something to consider until stumbling on it.

Will check a file with the “Name” tag set with skin.plextuary - 4.zip and upload a screenshot shortly.

skin.plextuary - 4.zip

“Name” tag:

No “Name” tag:

Perfect fix, IMO!

1 Like

Awesome, thanks for testing. Pushing another version. Don’t hate me, guys :smiley:

5 Likes

Hahaha, I’m sure everyone is loving the updates. Thank you :grinning_face:

Pushed skin.plextuarycpm: 4.0.0-pm4k1.3cpm.a14.3 to don’t panic repository

@jamal2367 Properly display subtitle name if set, as well; add scrolling to label · pannal/skin.plextuary@6beed27 · GitHub

5 Likes

Sorry, it must be documented somewhere, but how do I get from normal playback to that overlay?

Long press ENTER/OK

Edit: After clicking “Show stream info” (I think it was called) in the PM4K player settings

Edit 2: You should also be able to bind PlayerProcessInfo to any of your remote’s buttons via keymap editor

Edit 3: Make sure you’re on the latest beta

1 Like

I’m on 0.79-rev4.
I can’t find that in the player settings. In which on the sub menues (General, Video, UI, Player, Playback (user specific), Network, System?

Are you even using CoreELEC CPM? If not, there’s no way.

Show stream info can be found during playback in its settings (not inside the global settings)

No, I am not on CPM. Thank you for stating that.

Yes, the additional information being displayed is a benefit of using a CPM build. If you’d like to try:

PM4K 0.8.0-beta10.3

skin.plextuarycpm: 4.0.0-pm4k1.3cpm.a14.3

CPM A14

To see the “PlayerProcessInfo” screen, as pictured:

While playing a media file in PM4K, navigate to “Settings > Show Stream Info” within the on-screen display, then long press ENTER/OK.

or

Keymap Editor add-on within CoreELEC: Assign a remote button shortcut for the “Player Process Info” pane under “Global > Other”. Once assigned, pressing the assigned button while a media file is playing in PM4K will display the screen.

3 Likes

This is currently my change that looks the best so far.

<width>350</width>
<label>[UPPERCASE]$INFO[VideoPlayer.SubtitlesLanguage][/UPPERCASE] $INFO[VideoPlayer.SubtitlesName,| ,, (,)]</label>

This is not the end result.
I try to make it better.

It may be worth adding the “Name” tag to the “Audio language” section as well, in the use case of having a commentary audio track paired with a commentary subtitle, etc.

Also, I tested the scrolling setting for the “Subtitle language” section now that the “Name” tag is included. It does scroll, but does not seem to honor the column width:

Sorry for being a nuisance :grinning_face_with_smiling_eyes:

@fizzmissile
Set a width limit and activate scrolling.

<width>350</width>
<scroll>true</scroll>

Things should look much better after that.


I have made some changes for the subtitles + added a fallback.
I will commit it right away on github.
If @panni feels like it, he can take it over.

1 Like

Final result:


Let me explain the whole thing a bit.

Firstly, scrolling <scroll>true</scroll> is enabled and <width>350</width> is set to prevent characters from being too long..

If $INFO[VideoPlayer.SubtitlesLanguage] is not available, a fallback to $INFO[VideoPlayer.SubtitlesLangEx] is tried.

If $INFO[VideoPlayer.SubtitlesLanguage] is available, $INFO[VideoPlayer.SubtitlesLanguage] is taken as it should be.


If $INFO[VideoPlayer.SubtitlesName] is not available, no fallback is attempted and the line remains empty.

If $INFO[VideoPlayer.SubtitlesName] is available, $INFO[VideoPlayer.SubtitlesName] is used with | in first place.

This should cover everything.


@panni
Here is the commit if you are interested.

5 Likes

Thanks for info @jamal2367, the width setting did fix the scrolling width problem in the most current Plextuary CPM version from @panni should he choose to implement the fix:

I think for the sake of consistency, I do like @panni’s current implementation of using braces when the “Name” tag is present, and also keeping the fully-spelled portion of the language label (example: “English”) to match what’s displayed with “Audio language”. Non-scrolling examples of both here:

Plextuary CPM:


Estuary CPM:


I think it would also be a great feature if the “Name” tag was also displayed for the “Audio language” track. Say, in the above labeled instance if an audio commentary track was chosen and Name tagged “Commentary”:

Audio language: ENG | English (Commentary)
Subtitle language: ENG | English (Commentary)

or, with @jamal2367’s implementation which retains more display and scrolling space for the “Name” tag:

Audio language: ENG | Commentary
Subtitle language: ENG | Commentary

Just my two cents, of course. Maybe others can chime in. Thanks for both of your work in any case! :grinning_face:

I will not change anything in my version for the time being, as it contains a fallback and alternatives.

For audio, I don’t know if there’s anything more I can do, I want it to look good in the end.

1 Like

I took a bit of an amateur crack at it myself, and you may be right. Unless I’m missing something, with VideoPlayer.AudioName it seems to display both the audio track’s “Name” tag and additional audio codec information, which makes it all a bit messy in comparison to using VideoPlayer.SubtitlesName.

All the names of audio tracks and subtitles are available from the on-screen display in PM4K, so not a big deal to keep it clean and not surface this information in PlayerProcessInfo :+1:

Hi, panni. What’s the advantage of using path mapping to SMB over the http/https?

It may depend on your server. I tested running native server on synology and there was no difference. Others, on different servers had a change.

1 Like