[live] Weiterentwicklung v3.3.x

Zitat von MarkusE

Das ist sicher korrekt, aber man müsste dann live doch quasi neu schreiben (?).

Oder gibt es da da Alternativen?

Ich hatte nur mal über das JS hinter den Popups drübergeschaut und dachte mir "in jQuery dürfte das einfacher und besser wartbar sein".

Der Rest des JS (FB, Medienwiedergabe, Event/Aufnahmen-Logik) würde ich auch nicht antasten, selbst wenn ich in dem Thema drin wäre.

Bei mir ist es nämlich auch schon 10 Jahre her, als ich das letzte mal mit jQuery was gemacht habe - ich wäre da also auch keine wirkliche Hilfe bzw. gar ein Fix-Fertig-Patch-Ablieferer.

Ich bekomme leider mit der aktuellen git-Version keine Aufnahmen unter Aufnahmen angezeigt, auch bei den Serien und Sammlungen.

Unter Aufnahmen (flach) sind die Aufnahmen sichtbar.

Zitat von MarkusE

Lösche doch bitte mal den Browser Cache.

Bzw. Seite mit strg+F5 neu laden.

Browser Cache hatte ich schon prophylaktisch gelöscht und Browser neu gestartet.

Hat aber nicht geholfen.

Nachtrag: hier tntnet3.0

Zitat von MarkusE

Kannst Du mal mit Firefox testen?

Den Debugger von Firefox einschalten (F12), und schauen, ob es Fehlermeldungen gibt?

InstallTrigger sollte nicht mehr verwendet werden und wird in Zukunft entfernt werden.    constants.js:50:14
Uncaught (in promise) TypeError: obj is undefined                   recordings.html:241:9
    RecordingsSt_int http://192.168.139.150:8008/recordings.html?flat=false&filter=:241
    RecordingsSt_a http://192.168.139.150:8008/js/live/createHtml.js:309
    rec_string_d_a http://192.168.139.150:8008/js/live/createHtml.js:316
    Toggle http://192.168.139.150:8008/js/live/treeview.js:60
    onclick http://192.168.139.150:8008/recordings.html?flat=false&filter=:1

Zitat von MarkusE

Irgendwas im syslog, was den String "get_recordings" enthält?

Oder etwas mit "tntnet"?

Beide Strings gibt es nicht im syslog.

Ich habe mal --log=TRACE bei live eingestellt aber da kommt auch nicht mehr.

Das Einzige was im Zusammenhang mit live kommt ist Folgendes:

Nov 19 19:30:58 vdr vdr: [3979] live: DH: ------ RecordingsTree::RecordingsTree() --------, required time:   1,07090
Nov 19 19:30:58 vdr vdr: [3979] live: VDR-Live-Recordings-Tree-Open-Nodes = fldr_7CCB6B15,fldr_80A7D7B8

Zitat von MarkusE

Code

InstallTrigger sollte nicht mehr verwendet werden und wird in Zukunft entfernt werden.    constants.js:50:14

Das irritiert mich, das kommt nicht von live.

Kann sein, stand schon da bevor ich den Ordner aufgeklappt habe.

Zitat von MarkusE

Code

InstallTrigger sollte nicht mehr verwendet werden und wird in Zukunft entfernt werden.    constants.js:50:14

Das irritiert mich, das kommt nicht von live.

Das ist aus alten Firefox-XPI-Addon-Tagen, ignorieren.

Ich kann das Problem reproduzieren, habe aber keine Lösung.

Fakten:

- tritt nur mit tnt/cxx >= 3.0 auf

- tritt ab den commit auf

Somit schon mal eingegrenzt.

Ich habe mal meine alten tnt30 commit geprüft, was alles ab 3.0 nicht mehr geht. Dabei ist mir ein Punkt aufgefallen: Du darfst ab 3.0 keine Variable vom Type tChannelID mehr übergeben. Sie muss vorher in String gewandelt werden. Ist nur ein Verdacht, eine passende Stelle in dem o.g. Commit konnte ich noch nicht finden, der ist auch ganz schon groß.

Mein Verdacht mit tChannelID stimmt wohl nicht, ich habe aber was anderes gefunden:

Mit dem Patch:

diff --git a/live/js/live/treeview.js b/live/js/live/treeview.js
index f24d2f9..d61a8a0 100644
--- a/live/js/live/treeview.js
+++ b/live/js/live/treeview.js
@@ -49,7 +49,7 @@ class Treeview {
     this.folder_closed = folder_closed;
   }

-  async Toggle(node, node_id) {
+  Toggle(node, node_id) {
 // Unfold the branch if it isn't visible
     const sibling = findSibling(node, "UL");
     if (sibling == null) return;
@@ -57,7 +57,7 @@ class Treeview {
     if (sibling.style.display == 'none') {
       setImages(node, this.minus, this.folder_open);
       if (rec_ids[node_id] != null && rec_ids[node_id].length > 0) {
-        sibling.insertAdjacentHTML("beforeend", await rec_string_d_a(rec_ids[node_id]));
+        sibling.insertAdjacentHTML("beforeend", rec_string_d_a(rec_ids[node_id]));
           rec_ids[node_id] = [];
           if (typeof liveEnhanced !== 'undefined') liveEnhanced.domReadySetup();
         imgLoad();

Bekomme ich bei leeren Cache beim ersten Aufklappen eines Verzeichnis "[object Promise]" statt der Aufnahme Liste.

Wenn ich dann die Seite neu lade (refresh oder nochmals auf "Aufnahmen), dann kommt die Liste der Aufnahmen.

Ich hoffe, das hilft dir weiter.

Hatte wohl nichts mit meinen Vermutungen als Ursache zu tun. Muss wohl meine Glaskugel wieder zur Reparatur bringen.

Aber es funktioniert bei mir mit tnt/cxx 3.1.

Zitat von MarkusE

Im git ist ein update. Damit sollte auch tntnet 3 gehen

Vielen Dank!

Bei mir funktioniert es jetzt auch wieder.

Ich habe mal für meinen Hausgebrauch die Live-Icons in Anlehnung an die Originale in SVG zusammengestellt und leicht modernisiert:

Wäre das evtl. auch was, an dem andere User interessiert wären?

Ich meine, wenn mir das so zusagt, muss das ja noch lange nicht heißen, dass die Mehrheit das auch so sieht.

Allgemeiner Vorteil bei einer SVG-Integration wäre das unpixelige Aussehen auf Smartphones und anderen HiDPI-Displays.

Was ich geändert habe:

• Aufnehmen und Umschalten sind jetzt etwas selbsterklärender als die alten Symbole (bislang roter Knopf und Antennensymbol)
• Das Symbol für externe Wiedergabe ist selbsterklärender und unverwechselbarer: zeigt jetzt die Logos von VLC und MPV samt M3U drunter
• IMDb und Wikipedia haben jetzt ihre eigenen Symbole (letzteres nur als Idee, wird aktuell nicht genutzt)
• Man sieht es im Screenshot nicht, aber der rote Pfeil und das Ausrufezeichen sind SVG-Animationen

Ich bin aber auch noch nicht ganz fertig mit allen Symbolen.

Wobei man auch leichte Redundanzen und ein paar Symbole in dem Ordner erkennen kann, die schon seit längerem nicht mehr aktiv in Nutzung sind.

Aktuell konvertiere ich die SVG-Dateien manuell mit einem Skript mit Inkscape nach 16x16px-PNG und bügel die img-Ordner einfach über.

Was zu einer echten SVG-Integration notwendig wäre:

• Austauschen der Dateiendungen im Live-Quelltext
• CSS-Eigenschaft von aktuell width:16px auf width:5mm anpassen (wäre zumindest meine Empfehlung)
  Wobei man auf größeren Displays auch durchaus mal einen Ticken höher gehen könnte.

Ich bin auch für Verbesserungsvorschläge bei den Icons offen.

Unten mal ein Screenshot, wie die hochskalierten 16x16px-PNGs auf einem HiDPI-Desktop (WQXGA auf 16") "matschen".

Vertikal wäre sogar Platz für 24px/7.5mm - vergleiche dazu auch das unveränderte HD-Logo, das aktuell die Popup-Seitenleiste mit seinen 25px sprengt.

Hier mal noch der direkte Vergleich Original vs. SVG-Interpretation:

Edit: Kleinere Fixes und PNG-Vergleichsleiste hinzugefügt