[live] Weiterentwicklung v3.3.x

Kleiner Verbesserungsvorschlag, keine Ahnung ob umsetzbar:

wenn man in das Popup eines Eintrages geht, kann man Text zwar markieren. Sobald man die Maustaste aber loslässt, haut es die Markierung wieder weg. Ergo keine Chance auf Kopieren ins Clipboard oder auch nur "Google-Suche nach ..."

Edit:

Was mir mit aktivem tvscraper auch noch aufgefallen ist:

• die Thumbnails in der Programmübersicht könnten auch Links auf das Popup für den Eintrag sein
• die EPG-Sektion im Popup ist im Vergleich zum Scraper-Eintrag ja zumeist eher dünn
  Wäre toll wenn der Scraper-Eintrag bei strlen($scrapertext) > strlen($epgtext) dann Default-Anker oder an erster Stelle statt des EPG wäre. Oder wenn strlen($scrapertext)/strlen($epgtext) > 1.2, also z.B. mind. 20% länger.
• Links auf Bilder öffnen ja alle in einem neuem Tab
  man könnte die auch im selben Fenster formatfüllend einblenden (dann schließbar per klick auf Bild oder ein X rechts oben)

Edit2:

Noch einen kleinen CSS-Bug beim Popup bemerkt. Der Hintergrund-Verlauf bei EPG/Scraper/... scheint 1px zu tief zu hängen (FF unter Linux).

Erst mal vielen Dank für die Weiterentwicklung!

Ich hätte auch einen kleinen Verbesserungsvorschlag.

Ich wünsche mir einen Reset-Knopf für die Volltext-Suche in den Aufnahmen.

Der soll das Suchfeld leeren und ein Reload ausführen.

Vielleicht ist das ja ohne großen Aufwand machbar.

Gruß

Heiko

Zitat von heifisch

Ich wünsche mir einen Reset-Knopf für die Volltext-Suche in den Aufnahmen.

Der soll das Suchfeld leeren und ein Reload ausführen.

Nur das Suchfeld leeren oder das komplette Formular resetten?

Zitat von rüsseltier

Nur das Suchfeld leeren oder das komplette Formular resetten?

Jetzt muss man 3 Aktionen durchführen:

1. Cursor mit der Maus in das Suchfeld setzen,

2. Suchstring entfernen, mit Backspace oder was auch immer,

3. Enter drücken für ein reload,

Ich weiß, ich bin sehr faul.

Würde gerne nur mit der Maus auf eine Knopf drücken für diese Aktionen...

Was mir noch aufgefallen ist, in "Was läuft" könnte zwischen dem Datum und der Uhrzeit noch ein Leerzeichen rein.

Zitat von MarkusE

Keine Ahnung, warum sich der Browser so verhält. Du kannst aber den markieren Text mit der mittleren Maustaste an anderen Stellen wieder einfügen.

Du hast es bei Dir auch oder nicht?

Bei mir ist es auch nur auf das Popup beschränkt.

Bei der Programm-Liste in schedule.html kann man z.B. problemlos Unter Geiern Spielfilm, Deutschland/Frankreich 1964 rauskopieren.

Ich habe auch schon mal die JS-Dateien nach selection und mouse-Events durchsucht.

In den mootools gibt es wohl eine Art Mouseüberwachung, evtl. hängt es damit zusammen.

Ist aber auch nicht so wichtig...

Zitat

Ich glaube nicht, dass das für alle Anwender OK wäre. Gerade bei Verwendung von externem EPG.

Könnte gerne optional in den Einstellungen konfigurierbar sein.

Zitat

Wo genau? Ganz links?

Oberhalb EPG | Scraper | Schauspieler | Bilder ist die horizontale schwarze Umrandung zu hoch.

Deswegen ist der Hintergrundfarbverlauf sichtbar verrutscht.

Mal noch eine andere kleine Idee:

man könnte die Schauspielernamen in Richtung Wikipedia verlinken.

Also im Stile von "https://".$lang."wikipedia.org/wiki/".urlencode(str_replace(" ", "_", $actorname)).

Das wird zwar nicht immer klappen, aber doch in den meisten Fällen.

Hätte hier einen nahezu fertigen CPP-Analphabeten-Pullrequest für die Schauspieler-Links.

Es fehlt eigentlich nur noch die URL-Enkodierung des Schauspielernames mit vorherigem str.replace von Leerzeichen durch _.

Und die Ersetzung der Sprachen-Subdomain durch die ersten zwei Zeichen der genutzten Locale.

Nur so als Idee...

    actor_names.append("<a href=\"https://de.wikipedia.org/wiki/" + character->getPersonName() + "\" target=\"_blank\">" + character->getPersonName() + "</a>"); 

Zitat von MarkusE

Keine Ahnung, warum sich der Browser so verhält.

Die Zeiten, in denen in sattelfest in JS war, sind auch schon schon etwas her, aber könnte das Folgende in der infowin.js die Ursache sein?

      register: function(infoWin){
            this.fireEvent('onRegister', [infoWin]);
            infoWin.winFrame.addEvent('click', function(){
                    this.raise(infoWin);
                }.bind(this));
            infoWin.winFrame.inject(this.closedWins);
        },

Auf mich wirkt es, als ob bei jedem Klick ins InfoWin das InfoWin "geraised" wird, weswegen dann die Textauswahl verloren geht.

Bedeutet die Umstellung auf reines XMLHttpRequest eigentlich auch, dass Du Live zumindest teilweise auf ein zeitgenössisches JS-Framework umstellst?

Das Live-JS stammt ja noch aus einer Zeit, als man sich alles selbst "from scratch" zusammenklöppelte.

Während man das heute mit einem Framework vermutlich wesentlich eleganter hinbekäme.

Hi und Danke für die neue Version, bei mir läuft das kompilieren aber nicht durch unter archlinux.

cxxtools und tntnet sind Version 3.0:

yay -Q|grep -E 'cxx|tnt'

cxxtools 3.0-3

tntnet 3.0-3

Fehler:

LANG=C make PREFIX='/usr' && sudo make PREFIX='/usr' install && make clean

CC recman.o

In file included from /usr/include/vdr/i18n.h:14,

from /usr/include/vdr/config.h:19,

from /usr/include/vdr/channels.h:13,

from StringMatch.h:4,

from recman.cpp:2:

/usr/include/cxxtools/log/cxxtools.h:160:9: error: expected identifier before 'void'

160 | LOG_ERROR = 0x02,

| ^~~~~~~~~

compilation terminated due to -Wfatal-errors.

make: *** [Makefile:116: recman.o] Error 1

Ich kann das Problem mit cxxtools/tntnet V3.1 nicht reproduzieren, habe aber einen Verdacht. Versuche mal den Patch:

diff --git a/StringMatch.h b/StringMatch.h
index dd77581..f27a3cf 100644
--- a/StringMatch.h
+++ b/StringMatch.h
@@ -1,6 +1,9 @@
 #pragma once
 #ifdef HAVE_PCRE2
 #include <string>
+#if TNTVERSION >= 30000
+  #include <cxxtools/log.h>  // must be loaded before any vdr include because of duplicate macros (LOG_ERROR, LOG_DEBUG, LOG_INFO)
+#endif
 #include <vdr/channels.h>
 #include "stringhelpers.h"

kfb77

Danke - hat funktioniert!

MarkusE : Ich habe den Patch in dein git direkt in master eingestellt