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

DialogPVRChannelsOSD elements missing from on screen display (to master) #221

Merged
merged 1 commit into from Apr 6, 2017
Merged

DialogPVRChannelsOSD elements missing from on screen display (to master) #221

merged 1 commit into from Apr 6, 2017

Conversation

linknetx
Copy link
Contributor

@linknetx linknetx commented Mar 4, 2017

The progress bar and recording icon are not visible in the Live TV on screen channel display for a channel which is currently being recorded.

Suggest copying the focussed and item layouts completely (for easier future maintenance) from MyPVRChannels and adjusting the widths accordingly. The scrollbar position then also needs a slight adjustment.

The progress bar and recording icon are not visible in the Live TV on
screen channel display for a channel which is currently being recorded.

Suggest copying the focussed and item layouts completely (for easier
future maintenance) from MyPVRChannels and adjusting the widths
accordingly. The scrollbar position then also needs a slight adjustment.
@linknetx
Copy link
Contributor Author

linknetx commented Mar 7, 2017

Quote: in that case you should move the corresponding code to an include.

Sorry, but don't understand, what is an include?

Before focussed:
before 01

After focussed:
after 01

Before non-focussed:
before 2

After non-focussed:
after 02

@linknetx
Copy link
Contributor Author

linknetx commented Mar 28, 2017

@phil65 This PR must also be currently valid for the master version of your skin since your master still contains DialogPVRChannelsOSD.xml ?

@linknetx
Copy link
Contributor Author

@ksooo Your Sidenote for the Krypton PR:

Sidenote: Please be aware that in xbmc master DialogPVRChannelsOSD.xml recently got renamed to DialogPVRChannelGuide.xml. => xbmc/xbmc#11554

If I have understood it correctly it is DialogPVRGuideOSD.xml which has been renamed to DialogPVRChannelGuide.xml and not DialogPVRChannelsOSD.xml

Therefore this PR is still valid since DialogPVRChannelsOSD.xml has not been renamed and still has the bug.

@ksooo
Copy link
Contributor

ksooo commented Mar 28, 2017

If I have understood it correctly it is DialogPVRGuideOSD.xml which has been renamed to DialogPVRChannelGuide.xml and not DialogPVRChannelsOSD.xml

Ooops. My fault. You are right. :-)

@linknetx
Copy link
Contributor Author

@phil65 This is also valid for master branch, exactly the same code changes to DialogPVRChannelsOSD.xml as for the krypton branch.

Also DialogPVRChannelsOSD.xml has not been renamed see comments by @ksooo in previous comment.

@phil65 phil65 merged commit 8357fa8 into phil65:master Apr 6, 2017
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.

None yet

3 participants