129 replies to this topic

[catastrofus]

Als je in de HistoryZapSelector zit en je drukt op <menu> dan krijg je het onderstaande plaatje. Daar ontbreken de rode, groene, gele & blauwe 'balletjes'.

Op zich geen drama want de toetsen zijn wel bekend maar het hoort niet zo te zijn.

Is dit iets wat het PLi-team kan fixen of wordt dat een "contact the author of this plugin"?

Attached Files

•  zhb.jpg   98.08KB   10 downloads

[Frenske]

Dit zal (mogelijk) een gevolg zijn van de op handen zijnde grote wijzigingen in de GUI. Dat heeft op termijn gevolgen voor veel plugins maar het beoogde doel maakt dit hopelijk goed. 

[littlesat]

Zal een aanpassing in de code gemaakt moeten worden... door de labels de goede naam (key) te geven.

[catastrofus]

Dit is gewoon een standaard 7.3r plugin waarin bolletjes dus niet gedisplayed worden. Lijkt me niet dat deze plugin onderhavig is aan het probleem van "Dit zal (mogelijk) een gevolg zijn van de op handen zijnde grote wijzigingen in de GUI.".

Ik gebruik deze plugin al een tijdje en ik had zelf geen idee dat onder de <menu> knop ook nog een settings scherm zou verschijnen.

[Kaizoku]

When i click menu in history zap i get a popup settings menu not a skinned menu with minitv like you.

Attached Files

•  1_0_19_2459_440_1_C00000_0_0_0_20200813200715.png   1.12MB   7 downloads

[littlesat]

That might be because the plugin config screen is not named config.... and maybe has an instant embeded screen (why?)

[littlesat]

Do we have the sources of this plugin? It looks it has some unstructured stuff inside.... it looks like I can remove some lines and adapt it a bit and then with less it performs better...

Edited by littlesat, 13 August 2020 - 20:24.

[mrvica]

in class SetupZapSelectorScreen(Screen, ConfigListScreen):

replace this

<ePixmap pixmap="skin_default/buttons/green.png

<ePixmap pixmap="skin_default/buttons/red.png" 

<ePixmap pixmap="skin_default/buttons/blue.png"

with

<ePixmap pixmap="buttons/green.png

<ePixmap pixmap="buttons/red.png" 

<ePixmap pixmap="buttons/blue.png"

 

 

[catastrofus]

Volgens mij heb jij (of iemand anders, dat kan natuurlijk ook) hem aangepast aan de standaard PLi skins want vroeger zag deze er best wel anders uit.

 

Do we have the sources of this plugin? It looks it has some unstructured stuff inside.... it looks like I can remove some lines and adapt it a bit and then with less it performs better...

 

[littlesat]

Waarom ePixmap met instant embeded screen.... gewoon die class Config of Setup noemen of zo.... dan pakt hij gewoon het standaard Config scherm... dan kunnen ook meteen al die regels weg van dat embeded screen... door de class al een aparte naam te geven gaat het al mis...:

Edited by littlesat, 13 August 2020 - 21:36.

[hulkman]

Misschien is het een goed idee om een pli subteam te vormen dat zich uitsluitend bezig houdt met het aanpassen van applicatie skins in de standaard skin voor applicaties uit de feed. Zo zijn er ook heel wat die nog gebaseerd zijn op een sd skin, en daardoor nauwlijks leesbaar op wat kleiner scherm.
Het huidige systeem via aanmelden en de git-hub lujkt me niet erg productief gezien hier al jaren niets aan gedaan wordt.
Als je nu gewoon de aangepaste syntax kunt inleveren bij een moderator, en deze brengt het via de achterdeur in de skin. Uigteraard niet te veel fratsen e.d. maar gewoon SD naar HD conversie en kleine probleempjes oplossen zoals TS heeft aangegeven.

[littlesat]

Ik heb zo wie zo een patch klaarstaan die embeded skins kan aanpassen op de resolutie zonder aparte skins te doen voor elke resolutie in de code... maar voor setups is het gewoon slimmer voor een plug-in een al beschikbaar scherm wat geschikt is te gebruiken...

Edited by littlesat, 15 August 2020 - 12:57.

[hulkman]

Ik heb zo wie zo een patch klaarstaan die embeded skins kan aanpassen op de resolutie zonder aparte skins te doen voor elke resolutie in de code... maar voor setups is het gewoon slimmer voor een plug-in een al beschikbaar scherm wat geschikt is te gebruiken...

Ik begrijp dat je daarmee duidt op eventuele versie wijziging van een app?
Dat zal dan onderhouden moeten worden, maar als er niet meteen een passende skin voorhanden is, kun je tijdelijk terugvallen op het orgineel.
Voor je patch zie ik al helemaal niets in de weg staan, maar anderzijds zullen steeds meer app skins in HD res ontwikkeld gaan worden. Daar staat tegenover dat er ooit wel een 4K skin zal komen waar hetzelfde probleem zal optreden en je dezelfde truuk kunt toepassen.

[catastrofus]

Kan iemand dan, op een zeker moment, deze fix applyen?

 

 

 

in class SetupZapSelectorScreen(Screen, ConfigListScreen):

replace this

<ePixmap pixmap="skin_default/buttons/green.png

<ePixmap pixmap="skin_default/buttons/red.png" 

<ePixmap pixmap="skin_default/buttons/blue.png"

with

<ePixmap pixmap="buttons/green.png

<ePixmap pixmap="buttons/red.png" 

<ePixmap pixmap="buttons/blue.png"

 

 

 

 

[littlesat]

Totaal niet nodig.... al deze regels kunnen weg als je er voor zorgt dat de normale setup screen genomen wordt die elke skin heeft... en dan ook de labels voor de labels StaticText maken in de code en de juiste keys geven....

[Dimitrij]

1)Change the position of colored buttons setup FullHD skin

2)Delete old setup plugins History ZAP

[littlesat]

Why not compleet remove the embeded screen within the plugin and rely under the standard setup screen...... ??

Then all goes 'automatically' without any custom stuff....?

 

And then of course (not included below) give the correct keys to clear, ok, cancel..... "key_red, key_green, key_blue...."

 

Gone custom screen by using the standard what is already available... reduces also a lot of code and a crapy and ugly 'screenWidth check hack'. And it is in line what the skin 'says'.

 

I hope you now understand what I meen....

+class SetupZapSelectorScreen(Screen, ConfigListScreen, ProtectedScreen):
-class Setup(Screen, ConfigListScreen, ProtectedScreen):
-                global PLUGIN_VERSION
-                if screenWidth >= 1920:
-                               skin = """
-                               <screen position="center,center" size="765,550" >
-                                               <widget name="config" position="8,8" size="750,475" font="Regular;30" itemHeight="36" />
-                                               <ePixmap pixmap="skin_default/buttons/green.png" position="300,460" zPosition="0" size="210,60" alphatest="on" />
-                                               <ePixmap pixmap="skin_default/buttons/red.png" position="38,460" zPosition="0" size="210,60" alphatest="on" />
-                                               <ePixmap pixmap="skin_default/buttons/blue.png" position="570,460" zPosition="0" size="210,60" alphatest="on" />
-                                               <widget name="clear" position="540,450" size="210,60" valign="center" halign="center" zPosition="1" font="Regular;22" transparent="1" -backgroundColor="blue" />
-                                               <widget name="ok" position="270,450" size="210,60" valign="center" halign="center" zPosition="1" font="Regular;22" transparent="1" backgroundColor="green" />
-                                               <widget name="cancel" position="8,450" size="210,60" valign="center" halign="center" zPosition="1" font="Regular;22" transparent="1" -backgroundColor="red" />
-                               </screen>"""
-                else:
-                               skin = """
-                               <screen position="center,center" size="510,360" >
-                                               <widget name="config" position="5,5" size="500,310" />
-                                               <ePixmap pixmap="skin_default/buttons/green.png" position="180,300" zPosition="0" size="140,40" alphatest="on" />
-                                               <ePixmap pixmap="skin_default/buttons/red.png" position="5,300" zPosition="0" size="140,40" alphatest="on" />
-                                               <ePixmap pixmap="skin_default/buttons/blue.png" position="360,300" zPosition="0" size="140,40" alphatest="on" />
-                                               <widget name="clear" position="360,300" size="140,40" valign="center" halign="center" zPosition="1" font="Regular;17" transparent="1" backgroundColor="blue" />
-                                               <widget name="ok" position="180,300" size="140,40" valign="center" halign="center" zPosition="1" font="Regular;17" transparent="1" backgroundColor="green" />
-                                               <widget name="cancel" position="5,300" size="140,40" valign="center" halign="center" zPosition="1" font="Regular;17" transparent="1" backgroundColor="red" />
-                               </screen>"""
-
                def __init__(self, session, args=None):
-                               self.skin = SetupZapSelectorScreen.skin
                               self.setup_title = _("Setup Zap History:") + PLUGIN_VERSION
                               Screen.__init__(self, session)
                               ProtectedScreen.__init__(self)
-                               self["ok"] = Button(_("Save"))
-                               self["cancel"] = Button(_("Cancel"))
-                               self["clear"] = Button(_("Options"))
+                               self["key_green"] = StaticText(_("Save"))
+                               self["key_red"] = StaticText(_("Cancel"))
+                               self["key_blue"] = StaticText(_("Options"))

 

                               self["actions"] = ActionMap(["SetupActions", "ColorActions"],

Edited by littlesat, 18 August 2020 - 10:40.

[mrvica]

this works

from Components.Sources.StaticText import StaticText

class SetupZapSelectorScreen(Screen, ConfigListScreen):

	def __init__(self, session, args=None):
		self.setup_title = _("Setup Zap History:") + PLUGIN_VERSION
		Screen.__init__(self, session)
		
		self["key_green"] = StaticText(_("Save"))
		self["key_red"] = StaticText(_("Cancel"))
		self["key_blue"] = StaticText(_("Options"))
		self["actions"] = ActionMap(["SetupActions", "ColorActions"], 
		{
			"save": self.keyGreen,
			"cancel": self.keyRed,
			"blue": self.keyBlue,
		}, -2)
		ConfigListScreen.__init__(self, [])
		self.initConfig()
		self.createSetup()
		self.onClose.append(self.__closed)
		self.onLayoutFinish.append(self.__layoutFinished)

	def __closed(self):

are we talking about the same plugin, I don´t have here "Protected screen"

[littlesat]

And when you give the class a different name it is even better....

[mrvica]

I´ve tested it on fullNightHD, ok, on other skin config screen is not shown at all, we need some more investigation