nOpacity pre 1.0.0

lostinspc · 15. September 2013

Zitat von louis

Poste es doch mal, aber erst, nachdem du es angepasst hast

Die Icons sollte man dann wohl von darkred nehmen, die sind ja auch alle grau.

Ciao Louis

So, habs jetzt mal angepasst.

Icons nehme ich nicht, aber mit denen von darkred macht man aus meiner Sicht nichts falsch ...

Grüße, Peter

urknall · 15. September 2013

Mich stört das im Echtbetrieb sowieso nicht, da ich die Icons gar nicht verwende, ich hatte nur die aktuelle Git Version durchgetestet und die beiden Dinge sind mir aufgefallen...

louis · 15. September 2013

3PO: hm, den Sendernamen "reinquetschen" (genau das wäre es) möchte ich eigentlich nicht...ich bau das mit dem eigenen GHintergrund mal ein, wobei ich mir nicht sicher bin ob das was ausschaut, das macht irgendwie den Sinn der transparenten Icons zunichte...auf einen Hintergrund über das Script hast du keinen Bock?

Ciao Louis

PS: ich würde die große Schrift einen ticken kleiner machen...das überlappt. Oder brauchst du eine Brille?

3PO · 15. September 2013

Zitat von louis

[...] auf einen Hintergrund über das Script hast du keinen Bock?

Nun, das Script ist sicherlich nicht das Problem, sondern eher, dass dann die Logos in der Senderansicht auch einen Hintergrund haben und dann ist ja die ganze Opazität beim Teufel.

Zitat von louis

[...] PS: ich würde die große Schrift einen ticken kleiner machen...das überlappt. Oder brauchst du eine Brille?

Mit dieser Schriftgrösse nicht.

louis · 15. September 2013

Moin,

so...ich habe mal das darkgrey theme (thx @Peter) ins Git aufgenommen. Icontechnisch habe ich die von darkred (die grauen) benutzt und mit den Defaulticons in Blau gemischt, da rot ja nicht so wirklich dazu passt...

Ciao Louis

Ramirez · 15. September 2013

Da hier so viel über Farben diskutiert wird, mal eine Frage warum es zwei getrennte Arten gibt die Meldungen zu zeichen?

Sowohl in cNopacityDisplayMenuView::DrawMessage als auch in cNopacityDisplayMessage::SetMessage werden die Hintergrundfarben clrMessageStatus, clrMessageInfo, clrMessageWarning und clrMessageError genommen.

Aber der Text darauf wird einmal mit clrMenuFontMessages und einmal mit clrMessageFont.

Copperhead · 15. September 2013

Ich habe jetzt wegen dem doppelten Aufbau der Menüs mal weiter geschaut. Das passiert auch bei weiteren Ebenen in Plugin Einstellungsmenüs. Also in einer Ebene tiefer z.B. in den nOpacity Einstellungen.

lostinspc · 15. September 2013

Zitat von Copperhead

Ich habe jetzt wegen dem doppelten Aufbau der Menüs mal weiter geschaut. Das passiert auch bei weiteren Ebenen in Plugin Einstellungsmenüs. Also in einer Ebene tiefer z.B. in den nOpacity Einstellungen.

Also bei mir gibt's hier mit dem letzten git-checkout keine Aufälligkeiten

Copperhead · 15. September 2013

Bei mir scheinen das aber zwei unterschiedliche Probleme zu sein. Im Aufzeichnungsmenü bekomme ich es weg wenn ich das schmale Menü abschalte. Dann sind aber auch die Icons vom tvscraper weg.

louis · 15. September 2013

Zitat von Ramirez

Da hier so viel über Farben diskutiert wird, mal eine Frage warum es zwei getrennte Arten gibt die Meldungen zu zeichen?

Sowohl in cNopacityDisplayMenuView::DrawMessage als auch in cNopacityDisplayMessage::SetMessage werden die Hintergrundfarben clrMessageStatus, clrMessageInfo, clrMessageWarning und clrMessageError genommen.

Aber der Text darauf wird einmal mit clrMenuFontMessages und einmal mit clrMessageFont.

cNopacityDisplayMenuView::SetMessage() ist für die Ausgabe von Messages verantwortlich, wenn das VDR Menü offen ist. cNopacityDisplayMessage::SetMessage() wird aufgerufen, wenn kein OSD offen ist. Das erklärt auch die verschiedenen Theme Farben...die Benennung ist vielleicht ein bisschen "inkontinent"

Ciao Louis

louis · 15. September 2013

Copperhead: wie schonmal geschrieben ruft der VDR manchmal einfach zweimal das Menü auf, obwohl es nicht nötig ist. Ich habe mich darüber auch schon gewundert, als ich mit der Programmierung von nOpacity angefangen habe und mir das "Innenleben" etwas genauer angesehen habe. Warum das gemacht wird, das musst du Klaus fragen. Solange das Menü einfach gehalten ist, fällt das nicht auf. Wenn allerdings der Skin "fetter" ist und die CPU zusätzlich belastet ist, dann merkt man das. Deshalb mekrst du das auch nur in der schmalen Ansicht des Recordings Menüs, das ist wesentlich aufwändiger als das breite. Klaus hat da wohl an manchen Stellen ganz nach dem Prinzip "keep it simple" nicht so wirklich auf die Performance geachtet und das bewusst in Kauf genommen.

Ciao Louis

Copperhead · 15. September 2013

Gut und was mache ich jetzt dagegen?

louis · 15. September 2013

Im Setup Menü ist das übrgens ein anderer Effekt...dort wird das OSD bewusst geschlossen und wieder geöffnet, damit die geänderten Setup Einstellungen sofort ziehen.

Zitat von Copperhead

Gut und was mache ich jetzt dagegen?

Du hast verschiedene Möglichkeiten:
- damit leben
- Klaus fragen, ob er da noch etwas optimieren kann / will (ausser im Setup Menü, s.o.)
- einen schlanken Skin benutzen

Ciao Louis

Ramirez · 15. September 2013

Zitat von louis

cNopacityDisplayMenuView::SetMessage() ist für die Ausgabe von Messages verantwortlich, wenn das VDR Menü offen ist. cNopacityDisplayMessage::SetMessage() wird aufgerufen, wenn kein OSD offen ist. Das erklärt auch die verschiedenen Theme Farben...die Benennung ist vielleicht ein bisschen "inkontinent"

Ciao Louis

Das habe ich ja verstanden. Es war ja nur als Vereinfachungsvorschlag gedacht, um eine Farbe wegfallen zu lassen. Weil wohl jedes Theme diese beiden Einträgen den gleichen Wert zuordnet.

louis · 15. September 2013

Zitat von Ramirez

Das habe ich ja verstanden. Es war ja nur als Vereinfachungsvorschlag gedacht, um eine Farbe wegfallen zu lassen. Weil wohl jedes Theme diese beiden Einträgen den gleichen Wert zuordnet.

Na dann schreib das doch gleich, dann hätte ich mir das Posting sparen können

Ciao Louis

S:oren · 15. September 2013

Zitat von louis

Ich benutze die RSS Feeds im Skin selbst auch nicht mehr...war ein Schuss in den Ofen. Eigentlich sollte ich das komplett wieder rausnehmen und ein eigenes Mini Plugin draus machen.

Faende ich gut: weniger build-Abhaengigkeiten fuer skinnopacity...

Gruss,
S:oren

Saman · 16. September 2013

Moin Louis,

danke für den neuen Vollbild-Modus, gefällt mir sehr gut!
Habe das bei mir allerdings ein bisschen abgeändert, damit die Poster oben bündig mit dem Inhalt angezeigt werden.
Das alternative Aufnahme Bild lasse ich dazu noch im Posterformat (h=w*1.43) darstellen:

Vollbild

Diff

diff -ruN skinnopacity-git/imageloader.c skinnopacity/imageloader.c
--- skinnopacity-git/imageloader.c	2013-09-15 16:48:18.684134291 +0200
+++ skinnopacity/imageloader.c	2013-09-16 09:20:26.948248155 +0200
@@ -155,6 +155,20 @@
     return false;
 }
 
+bool cImageLoader::LoadRecordingImage(cString Path, int width, int height) {
+    if ((width == 0)||(height==0))
+        return false;
+    cString recImage("");
+    if (FirstImageInFolder(Path, "jpg", &recImage)) {
+        recImage = cString::sprintf("/%s", *recImage);
+        if (!LoadImage(*recImage, Path, "jpg"))
+            return false;
+        buffer.sample( Geometry(width, height));
+        return true;
+    }
+    return false;
+}
+
 bool cImageLoader::LoadAdditionalRecordingImage(cString path, cString name) {
     int width = config.epgImageWidthLarge;
     int height = config.epgImageHeightLarge;
diff -ruN skinnopacity-git/imageloader.h skinnopacity/imageloader.h
--- skinnopacity-git/imageloader.h	2013-09-15 16:48:18.684134291 +0200
+++ skinnopacity/imageloader.h	2013-09-16 09:20:26.993841907 +0200
@@ -20,6 +20,7 @@
     bool LoadEPGImage(int eventID);
     bool LoadAdditionalEPGImage(cString name);
     bool LoadRecordingImage(cString Path);
+    bool LoadRecordingImage(cString Path, int width, int height);
     bool LoadAdditionalRecordingImage(cString path, cString name);
     bool LoadPoster(const char *poster, int width, int height);
     void DrawBackground(tColor back, tColor blend, int width, int height, bool mirror = false);
diff -ruN skinnopacity-git/textwindow.c skinnopacity/textwindow.c
--- skinnopacity-git/textwindow.c	2013-09-15 16:48:18.688134291 +0200
+++ skinnopacity/textwindow.c	2013-09-16 10:18:20.817047516 +0200
@@ -63,8 +63,8 @@
             }
         } else {
             hasPoster = false;
-            posterHeight = 0;
-            posterWidth = 0;
+            posterWidth = geometry->Width() / 4;
+            posterHeight = (double)posterWidth * 1.43;
         }
     }
 }
@@ -178,34 +178,35 @@
     int border = config.borderDetailedEPG;
     int width = geometry->Width();
     int height = geometry->Height();
-    int widthTextHeader = width - 2 * border;
-    int widthText = widthTextHeader;
+    int widthTextSmall = width - 2 * border;
+    int widthText = widthTextSmall;
     int y = border;
+
+    //Title
+    y = DrawTextWrapper(event->Title(), widthText, y, border, fontHeader, Theme.Color(clrMenuFontDetailViewHeaderTitle));
+    //Short Text
+    y = DrawTextWrapper(event->ShortText(), widthText, y, border, font, Theme.Color(clrMenuFontDetailViewHeader));
+
+    y += fontHeader->Height();
+    //Image
     cImageLoader imgLoader;
     bool epgImageFound = false;
     if (hasPoster) {
         if (imgLoader.LoadPoster(poster.media.path.c_str(), posterWidth, posterHeight)) {
             int posterX = width - posterWidth - border;
             pixmap->DrawImage(cPoint(posterX, border), imgLoader.GetImage());
-            widthTextHeader -= posterWidth;
+            widthTextSmall -= posterWidth;
         }
     } else if (imgLoader.LoadEPGImage(event->EventID())) {
         epgImageFound = true;
         pixmap->DrawImage(cPoint(width - config.epgImageWidth - border, y), imgLoader.GetImage());
-        widthTextHeader -= config.epgImageWidth;
+        widthTextSmall -= config.epgImageWidth;
     }
-    //Title
-    y = DrawTextWrapper(event->Title(), widthTextHeader, y, border, fontHeader, Theme.Color(clrMenuFontDetailViewHeaderTitle));
-    //Short Text
-    y = DrawTextWrapper(event->ShortText(), widthTextHeader, y, border, font, Theme.Color(clrMenuFontDetailViewHeader));
-
-    y += fontHeader->Height();
     //Description
     int maxHeight = height - y;
     if (hasPoster && (y < (border + posterHeight))) {
-        int heightNarrow = border + posterHeight - y;
         DrawTextWrapperFloat(event->Description(), 
-                             widthTextHeader, widthText, y, heightNarrow,
+                             widthTextSmall, widthText, y, posterHeight,
                              border, font, Theme.Color(clrMenuFontDetailViewText), maxHeight);
     } else if (epgImageFound && (y < (border + config.epgImageHeight))) {
         y = border + config.epgImageHeight;
@@ -222,22 +223,10 @@
     int border = config.borderDetailedRecordings;
     int width = geometry->Width();
     int height = geometry->Height();
-    int widthTextHeader = width - 2 * border;
-    int widthText = widthTextHeader;
+    int widthTextSmall = width - 2 * border;
+    int widthText = widthTextSmall;
     int y = border;
-    cImageLoader imgLoader;
-    bool recImageFound = false;
-    if (hasPoster) {
-        if (imgLoader.LoadPoster(poster.media.path.c_str(), posterWidth, posterHeight)) {
-            int posterX = width - posterWidth - border;
-            pixmap->DrawImage(cPoint(posterX, border), imgLoader.GetImage());
-            widthTextHeader -= posterWidth;
-        }
-    } else if (imgLoader.LoadRecordingImage(recording->FileName())) {
-        pixmap->DrawImage(cPoint(width - config.epgImageWidth - border, y), imgLoader.GetImage());
-        widthTextHeader -= config.epgImageWidth;
-        recImageFound = true;
-    }
+
     const cRecordingInfo *info = recording->Info();
     if (!info)
         return;
@@ -247,22 +236,37 @@
     else    
         recTitle = recording->Name();
     //Title
-    y = DrawTextWrapper(*recTitle, widthTextHeader, y, border, fontHeader, Theme.Color(clrMenuFontDetailViewHeaderTitle));
+    y = DrawTextWrapper(*recTitle, widthText, y, border, fontHeader, Theme.Color(clrMenuFontDetailViewHeaderTitle));
     //Short Text
     if (!isempty(info->ShortText())) {
-        y = DrawTextWrapper(info->ShortText(), widthTextHeader, y, border, font, Theme.Color(clrMenuFontDetailViewHeader));
+        y = DrawTextWrapper(info->ShortText(), widthText, y, border, font, Theme.Color(clrMenuFontDetailViewHeader));
     }
     y += fontHeader->Height();
+    //Image
+    cImageLoader imgLoader;
+    bool recImageFound = false;
+    if (hasPoster) {
+        if (imgLoader.LoadPoster(poster.media.path.c_str(), posterWidth, posterHeight)) {
+            int posterX = width - posterWidth - border;
+            pixmap->DrawImage(cPoint(posterX, y), imgLoader.GetImage());
+            widthTextSmall -= posterWidth + border;
+        }
+    } else if (imgLoader.LoadRecordingImage(recording->FileName(), posterWidth, posterHeight)) {
+        int posterX = width - posterWidth - border;
+        pixmap->DrawImage(cPoint(posterX, y), imgLoader.GetImage());
+        widthTextSmall -= posterWidth + border;
+        recImageFound = true;
+    }
     //Description
     int maxHeight = height - y;
     if (hasPoster && (y < (border + posterHeight))) {
-        int heightNarrow = border + posterHeight - y;
         DrawTextWrapperFloat(recording->Info()->Description(), 
-                             widthTextHeader, widthText, y, heightNarrow,
+                             widthTextSmall, widthText, y, posterHeight,
+                             border, font, Theme.Color(clrMenuFontDetailViewText), maxHeight);
+    } else if (recImageFound && (y < (border + posterHeight))) {
+        DrawTextWrapperFloat(recording->Info()->Description(), 
+                             widthTextSmall, widthText, y, posterHeight,
                              border, font, Theme.Color(clrMenuFontDetailViewText), maxHeight);
-    } else if (recImageFound && (y < (border + config.epgImageHeight))) {
-        y = border + config.epgImageHeight;
-        DrawTextWrapper(recording->Info()->Description(), widthText, y, border, font, Theme.Color(clrMenuFontDetailViewText), maxHeight);
     } else {
         DrawTextWrapper(recording->Info()->Description(), widthText, y, border, font, Theme.Color(clrMenuFontDetailViewText), maxHeight);
     }
@@ -345,7 +349,7 @@
 void cNopacityTextWindow::DrawPoster(int border) {
     cImageLoader imgLoader;
     if (imgLoader.LoadPoster(poster.media.path.c_str(), posterWidth, posterHeight)) {
-        pixmap->DrawImage(cPoint(border, font->Height()), imgLoader.GetImage());
+        pixmap->DrawImage(cPoint(border, 2*border), imgLoader.GetImage());
     }
 }

Alles anzeigen

Hier ein Vorschlag um den transparenten Streifen bei den Scrollbars aufzufüllen:

Scrollbar

Diff

diff -ruN skinnopacity-git/displaymenuview.c skinnopacity/displaymenuview.c
--- skinnopacity-git/displaymenuview.c	2013-09-15 16:48:18.604134292 +0200
+++ skinnopacity/displaymenuview.c	2013-09-16 08:42:46.884338102 +0200
@@ -488,7 +488,7 @@
 
     if (config.menuAdjustLeft) {
         //Background
-        pixmapContent->DrawRectangle(cRect(0, 0, contentWidthFull - radius, contentHeight), Theme.Color(clrMenuBack));
+        pixmapContent->DrawRectangle(cRect(0, 0, osdWidth, contentHeight), Theme.Color(clrMenuBack));
         //Upper and lower Corner Square
         pixmapContent->DrawRectangle(cRect(contentWidthFull - radius, 0, radius, radius), Theme.Color(clrMenuBack));
         pixmapContent->DrawRectangle(cRect(contentWidthFull - radius, contentHeight - radius, radius, radius), Theme.Color(clrMenuBack));
@@ -501,14 +501,14 @@
         if (radius-borderWidth > 2) {
             //Upper Ellipse
             pixmapContent->DrawEllipse(cRect(contentWidthFull - radius, 0, radius, radius), Theme.Color(clrMenuBorder),2);
-            pixmapContent->DrawEllipse(cRect(contentWidthFull - radius + borderWidth, borderWidth, radius-borderWidth, radius-borderWidth), clrTransparent, 2);
+            pixmapContent->DrawEllipse(cRect(contentWidthFull - radius + borderWidth, borderWidth, radius-borderWidth, radius-borderWidth), Theme.Color(clrMenuBack), 2);
             //Lower Ellipse
             pixmapContent->DrawEllipse(cRect(contentWidthFull - radius, contentHeight - radius, radius, radius), Theme.Color(clrMenuBorder),3);
-            pixmapContent->DrawEllipse(cRect(contentWidthFull - radius + borderWidth, contentHeight - radius, radius-borderWidth, radius-borderWidth), clrTransparent, 3);
+            pixmapContent->DrawEllipse(cRect(contentWidthFull - radius + borderWidth, contentHeight - radius, radius-borderWidth, radius-borderWidth), Theme.Color(clrMenuBack), 3);
         }
     } else {
         //Background
-        pixmapContent->DrawRectangle(cRect(osdWidth - contentWidthMinimum + radius, 0, contentWidthFull - radius, contentHeight), Theme.Color(clrMenuBack));
+        pixmapContent->DrawRectangle(cRect(osdWidth - contentWidthMinimum - widthScrollbar - 2*borderWidth, 0, osdWidth, contentHeight), Theme.Color(clrMenuBack));
         //Upper and lower Corner Square
         pixmapContent->DrawRectangle(cRect(osdWidth - contentWidthMinimum, 0, radius, radius), Theme.Color(clrMenuBack));
         pixmapContent->DrawRectangle(cRect(osdWidth - contentWidthMinimum, contentHeight - radius, radius, radius), Theme.Color(clrMenuBack));
@@ -521,10 +521,10 @@
         if (radius-borderWidth > 2) {
             //Upper Ellipse
             pixmapContent->DrawEllipse(cRect(osdWidth - contentWidthMinimum, 0, radius, radius), Theme.Color(clrMenuBorder),1);
-            pixmapContent->DrawEllipse(cRect(osdWidth - contentWidthMinimum, borderWidth, radius-borderWidth, radius-borderWidth), clrTransparent, 1);
+            pixmapContent->DrawEllipse(cRect(osdWidth - contentWidthMinimum, borderWidth, radius-borderWidth, radius-borderWidth), Theme.Color(clrMenuBack), 1);
             //Lower Ellipse
             pixmapContent->DrawEllipse(cRect(osdWidth - contentWidthMinimum, contentHeight - radius, radius, radius), Theme.Color(clrMenuBorder),4);
-            pixmapContent->DrawEllipse(cRect(osdWidth - contentWidthMinimum, contentHeight - radius, radius-borderWidth, radius-borderWidth), clrTransparent, 4);
+            pixmapContent->DrawEllipse(cRect(osdWidth - contentWidthMinimum, contentHeight - radius, radius-borderWidth, radius-borderWidth), Theme.Color(clrMenuBack), 4);
         }
     }
 }

Alles anzeigen

And last but not least: abschaltbare Posteranzeige bei den Kanalinfos und abschaltbare Logos bei den Timern im Hauptmenü:

Schalter

Diff

diff -ruN skinnopacity-git/config.c skinnopacity/config.c
--- skinnopacity-git/config.c	2013-09-15 16:48:18.600134292 +0200
+++ skinnopacity/config.c	2013-09-16 09:05:07.772251240 +0200
@@ -35,6 +35,7 @@
     resolutionIconSize = 100;
     statusIconSize = 64;
     progressCurrentSchedule = 0;
+    displayChannelPoster = 1;
     //Display Replay 
     replayHeight = 25;
     replayBorderVertical = 15;
@@ -326,6 +327,7 @@
     else if (strcmp(Name, "resolutionIconSize") == 0)      resolutionIconSize = atoi(Value);
     else if (strcmp(Name, "statusIconSize") == 0)          statusIconSize = atoi(Value);
     else if (strcmp(Name, "progressCurrentSchedule") == 0) progressCurrentSchedule = atoi(Value);
+    else if (strcmp(Name, "displayChannelPoster") == 0)    displayChannelPoster = atoi(Value);
     else if (strcmp(Name, "replayHeight") == 0)            replayHeight = atoi(Value);
     else if (strcmp(Name, "replayBorderVertical") == 0)    replayBorderVertical = atoi(Value);
     else if (strcmp(Name, "replayBorderBottom") == 0)      replayBorderBottom = atoi(Value);
diff -ruN skinnopacity-git/config.h skinnopacity/config.h
--- skinnopacity-git/config.h	2013-09-15 16:48:18.600134292 +0200
+++ skinnopacity/config.h	2013-09-16 09:00:57.787751965 +0200
@@ -60,6 +60,7 @@
         int resolutionIconSize;
         int statusIconSize;
         int progressCurrentSchedule;
+        int displayChannelPoster;
         //DisplayReplay
         int replayHeight;
         int replayBorderVertical;
diff -ruN skinnopacity-git/displaychannel.c skinnopacity/displaychannel.c
--- skinnopacity-git/displaychannel.c	2013-09-15 16:48:18.600134292 +0200
+++ skinnopacity/displaychannel.c	2013-09-16 09:00:57.602595936 +0200
@@ -691,7 +691,7 @@
                     int remaining = (int)(e->EndTime() - time(NULL))/60;
                     strSeen = cString::sprintf("-%d/%dmin", remaining, e->Duration()/60);
                 }
-                DrawPoster(e);
+                if (config.displayChannelPoster) DrawPoster(e);
             } else {
                 strSeen = cString::sprintf("%dmin", e->Duration()/60);
             }
diff -ruN skinnopacity-git/setup.c skinnopacity/setup.c
--- skinnopacity-git/setup.c	2013-09-15 16:48:18.688134291 +0200
+++ skinnopacity/setup.c	2013-09-16 09:07:54.598728743 +0200
@@ -95,6 +95,7 @@
     SetupStore("resolutionIconSize", config.resolutionIconSize);
     SetupStore("statusIconSize", config.statusIconSize);
     SetupStore("progressCurrentSchedule", config.progressCurrentSchedule);
+    SetupStore("displayChannelPoster", config.displayChannelPoster);
     SetupStore("replayHeight", config.replayHeight);
     SetupStore("replayBorderVertical", config.replayBorderVertical);
     SetupStore("replayBorderBottom", config.replayBorderBottom);
@@ -304,6 +305,9 @@
     titleStyle[2] = tr("no title");
     discUsageStyle[0] = tr("free time in hours");
     discUsageStyle[1] = tr("free space in GB");
+    showTimers[0] = trVDR("no");
+    showTimers[1] = trVDR("yes");
+    showTimers[2] = tr("small");
     Set();
 }
 
@@ -328,7 +332,7 @@
         Add(new cMenuEditIntItem(cString::sprintf("%s%s", *spacer, tr("Adjust Font Size - free")), &tmpNopacityConfig->fontDiskUsage, -20, 20));
         Add(new cMenuEditIntItem(cString::sprintf("%s%s", *spacer, tr("Adjust Font Size - percent")), &tmpNopacityConfig->fontDiskUsagePercent, -20, 20));
     }
-    Add(new cMenuEditBoolItem(tr("Display Timers"), &tmpNopacityConfig->showTimers));
+    Add(new cMenuEditStraItem(tr("Display Timers"), &tmpNopacityConfig->showTimers, 3, showTimers));
     if (tmpNopacityConfig->showTimers) {
         Add(new cMenuEditIntItem(cString::sprintf("%s%s", *spacer, tr("Maximum number of Timers")), &tmpNopacityConfig->numberTimers, 1, 10));
         Add(new cMenuEditIntItem(cString::sprintf("%s%s", *spacer, tr("Width of Timers (Percent of OSD Width)")), &tmpNopacityConfig->menuWidthRightItems, 5, 30));
@@ -526,6 +530,7 @@
     Add(new cMenuEditIntItem(tr("Adjust Font Size - EPG Infotext"), &tmpNopacityConfig->fontEPGSmallSize, -20, 20));
     Add(new cMenuEditIntItem(tr("Adjust Font Size - Channel Group"), &tmpNopacityConfig->fontChannelGroupSize, -20, 20));
     Add(new cMenuEditIntItem(tr("Adjust Font Size - Next/Prev Channel Group"), &tmpNopacityConfig->fontChannelGroupSmallSize, -20, 20));
+    Add(new cMenuEditBoolItem(tr("Display Banner / Poster"), &tmpNopacityConfig->displayChannelPoster));
     SetCurrent(Get(currentItem));
     Display();
 }
diff -ruN skinnopacity-git/setup.h skinnopacity/setup.h
--- skinnopacity-git/setup.h	2013-09-15 16:48:18.688134291 +0200
+++ skinnopacity/setup.h	2013-09-15 17:04:24.204198060 +0200
@@ -41,6 +41,7 @@
     protected:
         const char *titleStyle[3];
         const char *discUsageStyle[2];
+        const char *showTimers[3];
         void Set(void);
     public:
         cNopacitySetupMenuDisplayMain(cNopacityConfig *data);
diff -ruN skinnopacity-git/timers.c skinnopacity/timers.c
--- skinnopacity-git/timers.c	2013-09-15 16:48:18.688134291 +0200
+++ skinnopacity/timers.c	2013-09-15 17:07:30.865693536 +0200
@@ -165,14 +165,15 @@
 
 void cNopacityTimer::DrawLogo(void) {
     pixmapLogo->Fill(clrTransparent);
+    int showTimerLogo = (config.showTimers < 2) ? 1 : 0;
     int logoWidth = config.timersLogoWidth;
     int logoHeight = config.timersLogoHeight;
     const cChannel *Channel = timer->Channel();
     if (Channel && Channel->Name()) {
         cImageLoader imgLoader;
-        if (imgLoader.LoadLogo(Channel->Name(), logoWidth, logoHeight)) {
+        if (showTimerLogo && imgLoader.LoadLogo(Channel->Name(), logoWidth, logoHeight)) {
             pixmapLogo->DrawImage(cPoint((width - logoWidth)/2, 1), imgLoader.GetImage());
-        } else if (imgLoader.LoadLogo(*(Channel->GetChannelID().ToString()), logoWidth, logoHeight)) {
+        } else if (showTimerLogo && imgLoader.LoadLogo(*(Channel->GetChannelID().ToString()), logoWidth, logoHeight)) {
             pixmapLogo->DrawImage(cPoint((width - logoWidth)/2, 1), imgLoader.GetImage());
         } else {
             cTextWrapper channel;

Alles anzeigen

Dann ist mir beim Basteln noch aufgefallen:
Kanalinfo : wenn die Sendung überzieht ist die Fortschrittsanzeige zu lang
Kanalinfo - Gruppen : Pfeile werden oben abgeschnitten wenn Schriften zu groß sind
Menü > Textfenster (klein) : Poster überdecken unten die Schrift. Das ist oben aber schon behoben mit:
- pixmap->DrawImage(cPoint(border, font->Height()), imgLoader.GetImage());
+ pixmap->DrawImage(cPoint(border, 2*border), imgLoader.GetImage());

Eventuell ist ja was für dich dabei.

Gruß S.

Edit: und nochmal komplett
Edit2: der Patch gibt leider einen reject in timers.c da Windows-Zeilenumbruch...

louis · 16. September 2013

Moin,

und hier die nächsten Änderungen im Git...ich habe es geschafft, jetzt sind keine offenen Tickets mehr da

Code

- added rounded corners for message display
- fixed issue with gradient display for newer imagemagick versions
- added setup option to display (or not display) poster or fanart in channel display
- possibility to set manual poster for recordings. This manual poster has to
  be named as "cover_vdr.jpg" and has to be placed directly in the recording
  folder. For series it is also possible to place a common poster for all
  series in a folder two or three directories above the actual recording
  folders. If a manual poster is found, all tvscraper information is ignored.

Insbesondere der letzte Punkt sollte bitte mal ausgiebig getestet werden...das war ein bisschen tricky mit den ganzen Fällen mit lokalen Bildern und ohne und mit Bildern vom tvscraper und ohne Ich habe mich mehr oder weniger an dem Patch von Saman orientiert. Es wird nach einem "cover_vdr.jpg" im Aufnahmeverzeichnis gesucht, ist dieses Bild vorhanden, werden die tvscraper Infos (falls vorhanden) ignoriert und nur dieses Poster (sowohl in der Aufzeichnunsliste als auch in der Detailansicht) verwendet. Für Serienverzeichnisse suche ich entsprechend des Patches auch zwei bzw. drei Verzeichnisse höher als die eigentlichen Aufnahmeverzeichnisse nach dem cover_vdr.jpg.

In der Aufzeichnungsliste habe ich das Symbol vor den Zeiten rausgeschmissen, ist gar kein Cover vorhanden, wird das von TheChief mal vorgeschlagene Icon mit der Filmrolle als Dummy benutzt.

Jo...wie gesagt, mal ordentlich testen Ciao Louis

louis · 16. September 2013

Saman: hui, da hast du ja wieder ordentlich rumgeschraubt

Da hat sich jetzt dummerweise einiges überschnitten, nach meinem letzten Commit wird der Patch wohl kaum mehr funktionieren, da hat sich einiges geändert. Magst du nicht mal lieber ein paar Screenshots mit vorher / nachher posten, dann kann ich direkt sehen, ob ich m ir das genauer anschaue oder eher keinen Bock drauf habe

Ciao Louis

louis · 16. September 2013

Zitat von Saman

Edit2: der Patch gibt leider einen reject in timers.c da Windows-Zeilenumbruch...

Jetzt nicht mehr

nOpacity pre 1.0.0

Jetzt mitmachen!

Teilen