2041 replies to this topic

[DimitarCC]

I just added support for the graphical PVRstate icons.... 

 

Cool. Is it the same like in OE-A?

[Huevos]

I just added support for the graphical PVRstate icons.... 

I hope that is going to be compatible with all other distros.

<widget name="statusicon" position="42,110" size="25,25" pixmaps="icons/mp_play.png,icons/mp_pause.png,icons/mp_stop.png,icons/mp_forward.png,icons/mp_backward.png,icons/mp_slow.png" zPosition="5" alphatest="on"/>

This is what we have all been using for years and what is used in 3rd party skins too.

 

And don't forget there also needs to be "statusicon_summery" in the python code (in InfoBarGenerics).

Edited by Huevos, 28 November 2023 - 11:05.

[DimitarCC]

So for timers list....

 

Is that layout OK?

 

 timers_vertical.jpg   117.62KB   2 downloads

Edited by DimitarCC, 28 November 2023 - 11:18.

[littlesat]

Looks better indeed.... but it would be more helpfull when we get more items in the lest... e.g. 12 instead of 8...

 

This is what I tried for now...

 

 timerlist.jpg   109.54KB   2 downloads

Edited by littlesat, 28 November 2023 - 11:45.

[littlesat]

I did not did something for the summary yet (display) and I did do it for the PVRstate screen... I have no idea why you created it somehow double... So also doing it in the player extra with an let's guess extra config.... No idea why this is needed.

But we can discuss of course or you can help to get this also better....

 

I at least tried to get the code more compact as I saw in OpenVix.

 

Currently in this skin there are svgs and it seems they are not shown correctly (see snapshot from WanWizard before)

Edited by littlesat, 28 November 2023 - 11:42.

[DimitarCC]

Looks better indeed.... but it would be more helpfull when we get more items in the lest... e.g. 12 instead of 8...

 

Well maximum possible is 10 items on full screen. That is maximum because otherwise items cut the info.

 

P.S. did you see the screen path screenshots?

[littlesat]

I just added an example with 10 items.... that is fine!

[littlesat]

Thanks, I missed those screenpath snapshots... looks pretty good !!!!

 

I hope for now the PVRstate changes are sufficient unless I made use or the PVRstate screen as I thought it was a bad idea to 'double' these widgets....

 

In addition for timeshift we also need a solution as in timeshift we do not see anything here... By origian we had the infobar on the bottom and the timeshift PVRstate on top...

Edited by littlesat, 28 November 2023 - 11:48.

[Huevos]

Looks better indeed.... but it would be more helpfull when we get more items in the lest... e.g. 12 instead of 8...

Even full screen there would only be room for 10 with the data the timer list needs to show. So in FullHDNight it must look something like my attachment. And that is only 10 items at itemHeight="90".

Attached Files

•  1.jpg   140.04KB   2 downloads

[DimitarCC]

Thanks, I missed those screenpath snapshots... looks pretty good !!!!

 

OK then please accept the pull

 

https://github.com/O...igma2/pull/3848

 

So that to work is needed that component.

[littlesat]

Did you also think about the changes on the menu template... that we have more information and also the paths...?

[Huevos]

I did not did something for the summary yet (display) and I did do it for the PVRstate screen... I have no idea why you created it somehow double... So also doing it in the player extra with an let's guess extra config.... No idea why this is needed.

But we can discuss of course or you can help to get this also better....

 

I at least tried to get the code more compact as I saw in OpenVix.

 

Currently in this skin there are svgs and it seems they are not shown correctly (see snapshot from WanWizard before)

@Littlesat, it doesn't matter how you code it. That is your choice. But it needs to be compatible with the widget that has already been in use for many years. i.e. the name of the widget needs to be the same ("statusicon") and the order of the pixmaps needs to be the same (pixmaps="icons/mp_play.png,icons/mp_pause.png,icons/mp_stop.png,icons/mp_forward.png,icons/mp_backward.png,icons/mp_slow.png").

[littlesat]

Merge that screen header addon.... I like these style of addons...

[littlesat]

@Hueovos, the widget has the same name and the order of the picons is also the same...

 

For me I fully understand why I never merged it as it is coded with too many lines and repeatable code... here I always put a red flag... and try to get it more compact...

Edited by littlesat, 28 November 2023 - 11:53.

[DimitarCC]

Did you also think about the changes on the menu template... that we have more information and also the paths...?

 

I am still trying to find a suitable place for the path there.

 

But i dont think i will add more items in the page....It should be too tight imho.

[littlesat]

I already had a location for it... as far I can see for now it always fits.... see my suggestions. But when you find a better method please feel free. Unless it can still be considered to put everything in the 'gray' space and also make the main title smaller... so use the kind of the addon style.

 

I also put the menu items more closer so the path has space and then still you have 2 more options in the list.

Edited by littlesat, 28 November 2023 - 11:56.

[DimitarCC]

Well but it is on the place where the pager should be...

So if there are 3 pages and long path it will not fit

 

 

 

 

I also put the menu items more closer so the path has space and then still you have 2 more options in the list.

 

Will look at it.

Edited by DimitarCC, 28 November 2023 - 11:58.

[littlesat]

Will look at it. -> Thanks!

Edited by littlesat, 28 November 2023 - 11:59.

[littlesat]

Also a suggestion for the service info screen:

 

    <parameter name="ServiceInfoLeft" value="0,0,843,36"/>
    <parameter name="ServiceInfoRight" value="843,0,1696,36"/>

  <screen name="ServiceInfoSimple" position="fill" flags="wfNoBorder" backgroundColor="transparent">
    <panel name="Screen_1_1" />
    <widget name="infolist" position="125,150" size="1696,776" itemHeight="38" font="Regular;28" selectionDisabled="1" scrollbarMode="showOnDemand" />
  </screen>
  <screen name="ServiceInfo" position="fill" flags="wfNoBorder" backgroundColor="transparent">
    <panel name="ServiceInfoSimple" />
    <widget source="session.CurrentService" render="Label" position="386,900" size="1153,37" transparent="1" zPosition="1" foregroundColor="foreground" font="Regular;33" valign="top" halign="center">
      <convert type="ServiceInfo">HasHBBTV</convert>
    </widget>
  </screen>

[littlesat]

And there is indeed something obvious here as you can see it building-up.... But this is not always the fact....Something I do not see in our current skin... I'm afraid this is because of the svg files.... so I'm afraid we can better revert to png.

Edited by littlesat, 28 November 2023 - 12:18.