nOpacity 1.1.3 - wieder mit tvscraper support

louis · May 24, 2014 at 5:40 PM

Moin,

nachdem Martin dankenswerterweise die tvscraper API and die des scraper2vdr Plugins angepasst hat, habe ich die Unterstützung in der Version dafür in nOpacity auch wieder reingenommen. Dazu ist aber die Version 0.2.0 vom tvscraper Voraussetzung.

Für die Selbstbauer: Ein "make clean" vor dem "make install" kann nicht schaden, da ich das alte tvscraper Services File rausgeschmissen habe...da sollte man mal frisch bauen

Hier die Änderungen in der Version 1.1.3:

Code

Version 1.1.3


- Vertical channel logo alignment configurable in displaychannel
- added dedicated background pixmap in detailview
- added support for tvscraper again, tvscraper Version 0.2.0 needed
- fixed possible division by zero in displaychannel poster display
- updated theme darkred

Ciao Louis

_Martin_ · May 24, 2014 at 5:59 PM

Na dann bin ich mal gespannt ob noch Fehler auftauchen oder ob alles glatt geht, nun sollten ja vielleicht ein paar mehr Tester da sein aber bisher sieht ja alles gut aus.

Grüße
Martin

wayne · May 24, 2014 at 8:19 PM

Hab' grad' alles über testing-yavdr installiert und bin restlos begeistert.

Das habt' ihr richtig gut gemacht...

Saman · May 25, 2014 at 11:36 AM

Moin Louis,

ein kleiner Fix:

Display Spoiler

Diff

diff -ruN skinnopacity-orig/detailview.c skinnopacity/detailview.c
--- skinnopacity-orig/detailview.c	2014-05-25 10:02:57.201665881 +0200
+++ skinnopacity/detailview.c	2014-05-24 23:21:42.901281915 +0200
@@ -469,7 +469,7 @@
 
 void cNopacityEPGView::CheckEPGImages(void) {
     if (eventID > 0) {
-        for (int i=1; i <= config.GetValue("numAdditionalEPGPictures"); i++) {
+        for (int i=1; i <= 10; i++) {
             cString epgimage;
             if (config.epgImagePathSet) {
                 epgimage = cString::sprintf("%s%d_%d.jpg", *config.epgImagePath, eventID, i);
@@ -501,7 +501,7 @@
                         picsFound++;
                     }
                 }
-                if (picsFound >= config.GetValue("numAdditionalRecEPGPictures")) {
+                if (picsFound >= 10) {
                     break;
                 }
             }
diff -ruN skinnopacity-orig/imageloader.c skinnopacity/imageloader.c
--- skinnopacity-orig/imageloader.c	2014-05-25 10:02:57.357665874 +0200
+++ skinnopacity/imageloader.c	2014-05-24 20:01:33.038844268 +0200
@@ -158,12 +158,14 @@
             int strlen = fileName.length();
             if (strlen < 8)
                 continue;
-            if (!fileName.compare(strlen-8, 8, "_vdr.jpg"))
+            if (fileName.compare(strlen-8, 8, "_vdr.jpg") != 0)
                 continue;
             fileName = fileName.substr(0, strlen - 4);
             *recImage = fileName.c_str();
+            closedir(folder);
             return true;
         }
     }
+    closedir(folder);
     return false;
 }
\ Kein Zeilenumbruch am Dateiende.

Display More

Die Farben / Flächen für die Tabs im Detailview passen ihmo noch nicht so richtig.
Zum testen habe ich für den Header erstmal clrMenuDetailViewTabs genommen.
An dieser Stelle fände ich aber eine dritte Farbe aber Sinnvoll, also zB:
clrMenuDetailViewHead
clrMenuDetailViewBack
clrMenuDetailViewTabs

Mit dem Patch heben sich die Tabs jetzt vom Hintergrund ab.

Display Spoiler

Diff

diff -ruN skinnopacity-orig/detailview.c skinnopacity/detailview.c
--- skinnopacity-orig/detailview.c	2014-05-25 10:02:57.201665881 +0200
+++ skinnopacity/detailview.c	2014-05-24 23:21:42.901281915 +0200
@@ -84,7 +84,8 @@
     if (!pixmapHeaderLogo) {
         pixmapHeaderLogo = osd->CreatePixmap(5, cRect(x, y, width, headerHeight));
     }
-    pixmapHeader->Fill(Theme.Color(clrMenuDetailViewBack));
+    tColor bgColor = Theme.Color(clrMenuDetailViewTabs);
+    pixmapHeader->Fill(bgColor);
     pixmapHeaderLogo->Fill(clrTransparent);
     //Channel Logo
     int logoWidth = geoManager->channelLogoWidth;
@@ -100,9 +101,9 @@
     int yDateTime = border;
     int yTitle = (headerHeight - fontHeaderLarge->Height()) / 2;
     int ySubtitle = headerHeight - fontHeader->Height() - border;
-    pixmapHeader->DrawText(cPoint(xText, yDateTime), dateTime.c_str(), Theme.Color(clrMenuFontDetailViewHeader), Theme.Color(clrMenuDetailViewBack), fontHeader);
-    pixmapHeader->DrawText(cPoint(xText, yTitle), title.c_str(), Theme.Color(clrMenuFontDetailViewHeaderTitle), Theme.Color(clrMenuDetailViewBack), fontHeaderLarge);
-    pixmapHeader->DrawText(cPoint(xText, ySubtitle), subTitle.c_str(), Theme.Color(clrMenuFontDetailViewHeader), Theme.Color(clrMenuDetailViewBack), fontHeader);
+    pixmapHeader->DrawText(cPoint(xText, yDateTime), dateTime.c_str(), Theme.Color(clrMenuFontDetailViewHeader), bgColor, fontHeader);
+    pixmapHeader->DrawText(cPoint(xText, yTitle), title.c_str(), Theme.Color(clrMenuFontDetailViewHeaderTitle), bgColor, fontHeaderLarge);
+    pixmapHeader->DrawText(cPoint(xText, ySubtitle), subTitle.c_str(), Theme.Color(clrMenuFontDetailViewHeader), bgColor, fontHeader);
 }
 
 void cNopacityView::DrawTabs(void) {
@@ -110,6 +111,7 @@
         pixmapTabs = osd->CreatePixmap(4, cRect(0, y + height - tabHeight, width, tabHeight));
     }
     tColor bgColor = Theme.Color(clrMenuDetailViewTabs);
+    tColor fgColor = Theme.Color(clrMenuDetailViewBack);
     pixmapTabs->Fill(clrTransparent);
     pixmapTabs->DrawRectangle(cRect(0, 0, width, 2), bgColor);
     int numTabs = tabs.size();
@@ -122,12 +124,12 @@
         pixmapTabs->DrawEllipse(cRect(xCurrent, tabHeight - border/2, border/2, border/2), clrTransparent, -3);
         pixmapTabs->DrawEllipse(cRect(xCurrent + tabWidth - border / 2 - 2, tabHeight - border/2, border/2, border/2), clrTransparent, -4);
         if (tab == activeView) {
-            pixmapTabs->DrawRectangle(cRect(xCurrent + 2, 0, tabWidth - 6, tabHeight - border/2 - 1), clrTransparent);
-            pixmapTabs->DrawRectangle(cRect(xCurrent + border / 2, tabHeight - border/2 - 1, tabWidth - border, border/2 - 1), clrTransparent);
-            pixmapTabs->DrawEllipse(cRect(xCurrent + 2, tabHeight - border/2 - 2, border/2, border/2), clrTransparent, 3);
-            pixmapTabs->DrawEllipse(cRect(xCurrent + tabWidth - border / 2 - 4, tabHeight - border/2 - 2, border/2, border/2), clrTransparent, 4);
+            pixmapTabs->DrawRectangle(cRect(xCurrent + 2, 0, tabWidth - 6, tabHeight - border/2 - 1), fgColor);
+            pixmapTabs->DrawRectangle(cRect(xCurrent + border / 2, tabHeight - border/2 - 1, tabWidth - border, border/2 - 1), fgColor);
+            pixmapTabs->DrawEllipse(cRect(xCurrent + 2, tabHeight - border/2 - 2, border/2, border/2), fgColor, 3);
+            pixmapTabs->DrawEllipse(cRect(xCurrent + tabWidth - border / 2 - 4, tabHeight - border/2 - 2, border/2, border/2), fgColor, 4);
         }
-        pixmapTabs->DrawText(cPoint(xCurrent + (tabWidth - textWidth) / 2, 2 + (tabHeight - font->Height())/2), tabText.c_str(), Theme.Color(clrMenuFontDetailViewText), (tab == activeView) ? clrTransparent : bgColor, font);
+        pixmapTabs->DrawText(cPoint(xCurrent + (tabWidth - textWidth) / 2, 2 + (tabHeight - font->Height())/2), tabText.c_str(), Theme.Color(clrMenuFontDetailViewText), (tab == activeView) ? fgColor : bgColor, font);
         xCurrent += tabWidth;
     }
 }
@@ -147,7 +149,7 @@
         scrollable = true;
     }
     if (!pixmapContentBack) {
-        pixmapContentBack = osd->CreatePixmap(3, cRect(x, y + headerHeight, width, contentHeight + tabHeight));
+        pixmapContentBack = osd->CreatePixmap(3, cRect(x, y + headerHeight, width, contentHeight)); //  + tabHeight
         pixmapContentBack->Fill(Theme.Color(clrMenuDetailViewBack));
     }
     pixmapContent = osd->CreatePixmap(4, cRect(x, y + headerHeight, width, contentHeight), cRect(0, 0, width, drawPortHeight));

Display More

Für "Keep it Light" habe ich die Farben noch ein bisschen angepasst.
Wie funktioniert das jetzt genau, mit den 'themeconfigs' bzw. theme-light.conf

Edit: theme-light.conf angehängt

Danke für die neue Version!
Grüsse

louis · May 25, 2014 at 2:55 PM

Hi Saman,

danke für die Bugreports, ich schaue mir das mal an. Die Tabs wollte ich eh nochmal überarbeiten und ein bisschen deutlicher erkennbar machen...

Bzgl. der themespezifischen Configs: jedes Theme hat seine eigene Config. Zuerst werden die generellen Default Configs (vom Skin selbst vorgegeben) geladen. Danach wird geprüft, ob für das verwendete Theme eine themespezifische (globale) Configdatei vorliegt, in der können für einen Theme beliebige Defaultwerte vorgegeben werden, die die generellen Default Config Werte überschreiben. Diese themespezifische Default Config kann dann nochmals über das Setup Menüp vom User selbst spezifisch angepasst werden.

Es gibt schon eine Default Config für dein Theme (theme-light.conf). Diese Datei liegt unter <PLGCONFDIR>/themeconfigs/ (bzw. wird beim make install aus dem Sourceverzeichnis dahin kopiert). Schau dir die einfach mal an, da kannst du dann alle Default Vorgaben spezifisch für das Light Theme definieren. Jeder User, der dann das Theme light benutzt, hat diesen Default.

Wenn du mir eine angepasste theme-light.conf zukommen lässt, übernehme ich die natürlich gerne ins Git.

Ciao Louis

iNOB · May 25, 2014 at 4:41 PM

Die skinicons wurden farblich dem darkred Theme angepasst (rot/grau). Die ursprüngliche Farbgebung (weiß/grau) war für mich besser zu erkennen. Gibts die Icons auch noch irgendwo in weiß, oder zumindest als SVG Vorlage? Dann würde ich mir die selbst einfärben... ?

Gruß
iNOB

louis · May 25, 2014 at 5:10 PM

iNOB: welche skinicons meinst du? Mit welchem Theme? Ich habe eigentlich nur das darkred Theme auf den Stand vom Git von TheChief aktualisiert...ansonsten ist was schief gelaufen

Ciao Louis

TheChief · May 25, 2014 at 5:16 PM

Eventuell meinst du die Icons in der Kanalanzeige? Die hatte ich vor langer Zeit mal geändert, als einzelne Bilder dafür möglich waren. Glaub, die kann man auch löschen, dann wird die default-Grafik verwenden. Aber ich finds gut so und kann sie gut erkennen.

iNOB · May 25, 2014 at 5:21 PM

Ich habe die Icons (nur die, die mit letztem Git-Update geändert wurden) dummerweise aus dem darkred Theme in meinen eigenen "darkredspecial" kopiert. Jetzt sind die Anzeigen (rechts unten) in der Kanalanzeige rot statt weiß (ON=rot, OFF= grau).

TheChief
Zum einen Geschmackssache, zum Anderen eine Sache der eigenen Sehfähigkeiten. Manche haben halt mal ne rot o. grün Schwäche.

Gruß
iNOB

Saman · May 25, 2014 at 5:28 PM

Quote from louis

Wenn du mir eine angepasste theme-light.conf zukommen lässt, übernehme ich die natürlich gerne ins Git.

Super, ich habe eine theme-light.conf oben mit angehängt.

Hat ein bisschen gedauert, bis ich geschnallt hatte das du nur die zur theme-xxx.conf geänderten Werte in die setup.conf übernimmst

Grüsse

TheChief · May 25, 2014 at 6:08 PM

Quote from iNOB

Ich habe die Icons (nur die, die mit letztem Git-Update geändert wurden) dummerweise aus dem darkred Theme in meinen eigenen "darkredspecial" kopiert. Jetzt sind die Anzeigen (rechts unten) in der Kanalanzeige rot statt weiß (ON=rot, OFF= grau).

TheChief
Zum einen Geschmackssache, zum Anderen eine Sache der eigenen Sehfähigkeiten. Manche haben halt mal ne rot o. grün Schwäche.

Gruß
iNOB

Sorry, wollte da niemand zu nahe treten. Da es aber ursprünglich "mein" Theme war, war ich so frei, die Icons anzupassen. Man kann aber in Gimp oder PS sicher an den Reglern drehen und die wieder heller machen. Oder auf den git Stand vorher wechseln und die alten Bilder auschecken, müsste doch gehen, oder?

louis · May 25, 2014 at 6:26 PM

iNOB: lösch einfache die entsprechenden Icons von darkred, dann werden die default Icons genommen, das sind die weißen.

Ciao Louis

Ramirez · May 25, 2014 at 10:40 PM

Gerade mal meinen testing yaVDR upgedated. Funktioniert einwandfrei. Danke an Martin und louis

Keine Ahnung, ob es hier der richtige Thread ist, aber für nOpacity hätte ich noch einen kleinen Wunsch bezüglich der Menüicons:
Könnte man da ein (optionales) High Icon hinzufügen.

Hintergrund auf meinem Test VDR verwende ich das Default Theme und da sind ja die einfarbigen Icons in Verwendung. Man sieht auch, dass die Schriftfarbe für das aktive Menüelement wechselt. Die Icons bleiben aber weiß und sind auf dem gelben Hintergrund damit schlecht erkennbar.
Bin ich der einzige dem es so geht?

Frodo · May 25, 2014 at 10:50 PM

Was passiert wenn tvscraper und scraper2vdr gleichzeitig verwendet werden?

louis · May 25, 2014 at 11:00 PM

Quote from Frodo

Was passiert wenn tvscraper und scraper2vdr gleichzeitig verwendet werden?

Dann wird sich ein Krater öffnen und dich verschlucken

Spass bei Seite...dann gewinnt scraper2vdr

Ciao Louis

louis · May 25, 2014 at 11:04 PM

Ramirez: programmiertechnisch sind "highicons" kein Problem...müsste nur "jemand" die hellgrauen Icons dunkelgrau einfärben. Mach doch mal und stell nen Feature Request auf vdr-developer dafür ein

Ciao Louis

Frodo · May 26, 2014 at 8:29 AM

Quote from louis

Dann wird sich ein Krater öffnen und dich verschlucken

Spass bei Seite...dann gewinnt scraper2vdr

Ciao Louis

Super, dann muss man sich keine Sorgen machen wenn man tvscraper vergessen hat zu deaktivieren.
Vermutlich bedeutet das aber auch wenn beide aktiviert wurden und tvscraper2vdr nicht richtig konfiguriert wurde gibt es auch keine Bilder - das sollte man im Hinterkopf zur Fehlersuche behalten.

louis · May 26, 2014 at 8:57 AM

Quote from Frodo

Vermutlich bedeutet das aber auch wenn beide aktiviert wurden und tvscraper2vdr nicht richtig konfiguriert wurde gibt es auch keine Bilder - das sollte man im Hinterkopf zur Fehlersuche behalten.

Klar, wenn scraper2vdr aktiv ist, dann wird nur der angefragt und tvscraper ignoriert. Liefert dann scraper2vdr keine Bilder, werden auch keine angezeigt, obwohl tvscraper ggf. welche hätte.

Aber das macht ja keiner, jeder überlegt sich ja vorher eine für ihn sinnvolle Konstellation

Ciao Louis

Ramirez · May 26, 2014 at 6:22 PM

Quote from louis

Ramirez: programmiertechnisch sind "highicons" kein Problem...müsste nur "jemand" die hellgrauen Icons dunkelgrau einfärben. Mach doch mal und stell nen Feature Request auf vdr-developer dafür ein

Ciao Louis

Das klingt doch gut. Mache ich gerne. Da ich kein Graphiker bin, nun die Frage, wo kriege ich die korrekten Farbwerte her?
Laut meinem Graphikprogramm ist die hellgraue Farbe für die Icons momentan FFCCCCCC. Diesen Wert kann ich momentan nirgends in der nOpacity-default.theme finden. Wie komme ich auf den korrekten dunkelgrauen Wert?

TheChief · May 26, 2014 at 6:47 PM

Die Icons sind doch Grafiken, die wirst du nicht im Theme-file finden, sondern bei den menuIcons?!

nOpacity 1.1.3 - wieder mit tvscraper support

Participate now!

Users Viewing This Thread